github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-11-21 20:47:08 -05:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #1366 from sharelatex/ho-bump-compile-size

Browse source 
Bump compile size

GitOrigin-RevId: 087e8556a7e2e39bed486cdd8829bf427aaed68b
This commit is contained in:
Henry Oswald 2019-01-08 16:19:25 +00:00 committed by sharelatex
parent 563cdfd439
commit 68a0ff08ea
3 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
services/web/app/coffee/Features/Compile/ClsiFormatChecker.coffee
View file

@ -1,5 +1,6 @@
_ = require("lodash")
async = require("async")
settings = require("settings-sharelatex")
module.exports = ClsiFormatChecker =
@ -39,7 +40,7 @@ module.exports = ClsiFormatChecker =
_checkDocsAreUnderSizeLimit: (resources, callback)->
FIVEMB = 1000 * 1000 * 5
sizeLimit = 1000 * 1000 * settings.compileBodySizeLimitMb
totalSize = 0
@ -53,7 +54,7 @@ module.exports = ClsiFormatChecker =
totalSize += result.size
return result
tooLarge = totalSize > FIVEMB
tooLarge = totalSize > sizeLimit
if !tooLarge
return callback()
else

2
services/web/config/settings.defaults.coffee
View file

@ -488,6 +488,8 @@ module.exports = settings =
# currentImage: "texlive-full:2017.1"
# imageRoot: "<DOCKER REPOSITORY ROOT>" # without any trailing slash
compileBodySizeLimitMb: process.env['COMPILE_BODY_SIZE_LIMIT_MB'] or 5
# allowedImageNames: [
# {imageName: 'texlive-full:2017.1', imageDesc: 'TeXLive 2017'}

2
services/web/test/unit/coffee/Compile/ClsiFormatCheckerTests.coffee
View file

@ -8,7 +8,7 @@ SandboxedModule = require('sandboxed-module')
describe "ClsiFormatChecker", ->
beforeEach ->
@ClsiFormatChecker = SandboxedModule.require modulePath, requires:
"settings-sharelatex": @settings ={}
"settings-sharelatex": @settings = { compileBodySizeLimitMb: 5 }
"logger-sharelatex": @logger = { log: sinon.stub(), error: sinon.stub(), warn: sinon.stub() }
@project_id = "project-id"