Skip to content

Commit 4dc09b1

Browse files
author
Martii
committed
Make a VPS togglable option to set script storage to read-only
Applies to OpenUserJS#730
1 parent 8d01c69 commit 4dc09b1

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

controllers/scriptStorage.js

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,8 @@ var cleanFilename = require('../libs/helpers').cleanFilename;
1818
var findDeadorAlive = require('../libs/remove').findDeadorAlive;
1919
var userRoles = require('../models/userRoles.json');
2020

21+
var settings = require('../models/settings.json');
22+
2123
var parsers = (function () {
2224
return {
2325
UserScript: PEG.buildParser(fs.readFileSync('./public/pegjs/blockUserScript.pegjs', 'utf8'),
@@ -388,7 +390,7 @@ exports.storeScript = function (aUser, aMeta, aBuf, aCallback, aUpdate) {
388390
var libraries = [];
389391

390392

391-
if (!aMeta) {
393+
if (!aMeta || settings.read_only_script_storage) {
392394
return aCallback(null);
393395
}
394396

@@ -565,6 +567,11 @@ exports.webhook = function (aReq, aRes) {
565567

566568
aRes.end(); // Close connection
567569

570+
// Return if script storage is in read-only mode
571+
if (settings.read_only_script_storage) {
572+
return;
573+
}
574+
568575
// Test for known GH webhook ips: https://api.github.com/meta
569576
if (!aReq.body.payload ||
570577
!/192\.30\.25[2-5]\.(25[0-5]|2[0-4][0-9]|1[0-9]{2}|[1-9]?[0-9])$/

models/settings.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
{
22
"secret" : "someSecretStringForSession",
33
"connect" : "mongodb://dev:[email protected]:41651/openuserjs_dev",
4-
"maximum_upload_script_size": 1048576
4+
"maximum_upload_script_size": 1048576,
5+
"read_only_script_storage": false
56
}

0 commit comments

Comments
 (0)