krakenjs / lusca

Application security for express apps.
Other
1.79k stars 139 forks source link

Change whitelist/blacklist to allowlist/blocklist #140

Closed linkRace closed 3 years ago

linkRace commented 4 years ago

@grawk travis seems to be failing on old node versions for this, do you know who owns the config for those?

grawk commented 3 years ago

Hi @linkRace thanks for initiating this change.

Would you mind making a change to .travis.yml to select only node@10 and node@12 runtimes? The tests should pass with that change and we can move forward.

linkRace commented 3 years ago

Looks good now 👍

linkRace commented 3 years ago

Will merge this and submit another change before publishing as 1.7.0