diff options
| author | Yuchen Pei <hi@ypei.me> | 2022-10-13 13:55:41 +1100 |
|---|---|---|
| committer | Yuchen Pei <hi@ypei.me> | 2022-10-13 14:14:29 +1100 |
| commit | fcd816e95e73a78fd24a7e6baef709d7657d265f (patch) | |
| tree | 807494ee13cf6570f4c11fc29a8c64b7ab846a09 /common | |
| parent | 88fa677b51b6b1eb814866ea08129a6b7cda9ddb (diff) | |
`// @license` should be at the beginning of a line (mod whitespace)
Otherwise the following will be processed for @license / @license-end
method:
// foo.js
// // @license magnet:?xt=urn:btih:1f739d935676111cfff4b4693e3816e664797050&dn=gpl-3.0.txt GPL-3.0
// document.getElementById('bar');
// // @license-end
function foo(x, y) {
console.log(x + y);
}
$ node ./utitlities/check-script foo.js
[
true,
'// // @license magnet:?xt=urn:btih:1f739d935676111cfff4b4693e3816e664797050&dn=gpl-3.0.txt GPL-3.0\n' +
"// document.getElementById('bar');\n" +
'// // @license-end\n' +
'function foo(x, y) {\n' +
' console.log(x + y);\n' +
'}\n',
'\n' +
'Recognized license: "GNU General Public License (GPL) version 3".\n' +
'Script appears to be trivial.'
]
Diffstat (limited to 'common')
| -rw-r--r-- | common/checks.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/common/checks.js b/common/checks.js index 2a4ab9c..01765e4 100644 --- a/common/checks.js +++ b/common/checks.js @@ -56,8 +56,8 @@ const LOOPKEYS = new Set(['for', 'if', 'while', 'switch']); const OPERATORS = new Set(['||', '&&', '=', '==', '++', '--', '+=', '-=', '*']); // @license match, second and third capture groups are canonicalUrl // and license name -const OPENING_LICENSE_RE = /\/[/*]\s*?(@license)\s+(\S+)\s+(\S+).*$/mi; -const CLOSING_LICENSE_RE = /\/([*/])\s*@license-end\s*(\*\/)?/mi; +const OPENING_LICENSE_RE = /^\s*\/[/*]\s*?(@license)\s+(\S+)\s+(\S+).*$/mi; +const CLOSING_LICENSE_RE = /^\s*\/([*/])\s*@license-end\s*(\*\/)?/mi; /** * If this is true, it evaluates entire scripts instead of returning as soon as it encounters a violation. * |