Skip to content

Conversation

tofumatt
Copy link
Contributor

@tofumatt tofumatt commented Dec 4, 2015

Part of #66, this checks all JS files against a hash of known JS
libraries.

We're borrowing the current list from the old amo-validator
but in the future (#384) we'll have our own list.

Named filehasher and avoiding the name whitelist because of @andymckay's
thoughtful blog post.

@muffinresearch
Copy link
Contributor

By a miss, do you mean it not flagging when there are no matches?

Yep exactly - just showing that a non-matching file isn't in the list.

@muffinresearch
Copy link
Contributor

r+wc

Part of #66, this checks all JS files against a hash of known JS
libraries.

We're borrowing the current list from the old [amo-validator](https://github.com/mozilla/amo-validator/blob/master/validator/testcases/hashes.txt)
but in the future (#384) we'll have our own list.

Named filehasher and avoiding the name whitelist because of @andymckay's
[thoughtful blog post](http://mckay.pub/2015-01-07-allowed/).
@tofumatt tofumatt force-pushed the check-for-known-js-libs-383 branch from 127c275 to 722351b Compare December 4, 2015 23:00
tofumatt added a commit that referenced this pull request Dec 4, 2015
Compare JS files against known JS libs (fix #383)
@tofumatt tofumatt merged commit d4f6600 into master Dec 4, 2015
@tofumatt tofumatt deleted the check-for-known-js-libs-383 branch December 4, 2015 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants