Closed GabLeRoux closed 6 years ago
You should remove unnecessary files from the addon pack. Then use web-ext lint to verification.
Thanks, I'll definitely try this when I get some more time.
Since I already have eslint running, I suspect it's the bower_component
folder that is causing this. (I didn't have a look at the report yet. Had the error and went to bed).
Maybe generating the whole addon
from webpack
instead of bundling it with the complete bower_component
could fix the problem by removing the only unnecessary files.
I tried to bundle everything with webpack, but I had troubles with datatables
, jquery
and the datatables bootstrap. I'll have to try harder, but npm's version didn't seem up to date compared to bower's version.
A workaround I see could be to use a cdn instead so I could avoid bower_components
completely. Would be a reasonable solution for first version, but I'd prefer having a plugin that works even without internet connection.
Concerning the web-ext lint, I suppose it shouldn't be too hard to include this in de dev pipeline in webpack. I even plan to do the extension release creation from travis later.
Thanks for the hints :)
I believe it could be very cool to submit this plugin ! There is actually no cool plugin who replaced Norwell ! And there was a lot of user who used it !
:-) !
I'm looking into this right away, I'll try a bit more with webpack
and web-ext lint
as I noted above and will update here
So cool ! Thanks !
As I was suspecting, all of the lint errors happen in addon/bower_components
folder:
npm i -g web-ext
npm i && bower i
npm run build
web-ext lint -s=addon
Currently trying to get rid of bower_components
and bundle everything myself with webpack
and node_modules
. Will time frame this to 1h. If I'm not done with it after this, I'll send my work in a branch will use a CDN for the release.
I think I'll be ok with #5 in place, I ran web-ext lint
again and I don't have errors anymore. Only a few warnings and notices which are probably all fine. Expect a version on addons.mozilla.org soon :)
npm run lint
> webextensions-history-browser@1.0.0 lint /Users/gableroux/repos/firefox-extensions/webextensions-history-browser
> eslint . && web-ext lint -s=addon
Validation Summary:
errors 0
notices 1
warnings 8
NOTICES:
Code Message Description File Line Column
MOZILLA_COND_OF_USE Violation of Mozilla conditions of use. Words found that violate the Mozilla conditions of use. See js/bundle.js
https://www.mozilla.org/en-US/about/legal/acceptable-use/ for more details.
WARNINGS:
Code Message Description File Line Column
UNSAFE_VAR_ASSIGNMENT Unsafe assignment to innerHTML Due to both security and performance concerns, this may not be set using dynamic values which have js/bundle.js 5829 4
not been adequately sanitized. This can lead to security issues or fairly serious performance
degradation.
UNSAFE_VAR_ASSIGNMENT Unsafe assignment to innerHTML Due to both security and performance concerns, this may not be set using dynamic values which have js/bundle.js 9338 5
not been adequately sanitized. This can lead to security issues or fairly serious performance
degradation.
UNSAFE_VAR_ASSIGNMENT Unsafe assignment to innerHTML Due to both security and performance concerns, this may not be set using dynamic values which have js/bundle.js 10549 8
not been adequately sanitized. This can lead to security issues or fairly serious performance
degradation.
UNSAFE_VAR_ASSIGNMENT Unsafe assignment to innerHTML Due to both security and performance concerns, this may not be set using dynamic values which have js/bundle.js 29604 4
not been adequately sanitized. This can lead to security issues or fairly serious performance
degradation.
UNSAFE_VAR_ASSIGNMENT Unsafe assignment to innerHTML Due to both security and performance concerns, this may not be set using dynamic values which have js/bundle.js 29820 6
not been adequately sanitized. This can lead to security issues or fairly serious performance
degradation.
UNSAFE_VAR_ASSIGNMENT Unsafe assignment to innerHTML Due to both security and performance concerns, this may not be set using dynamic values which have js/bundle.js 31230 7
not been adequately sanitized. This can lead to security issues or fairly serious performance
degradation.
UNSAFE_VAR_ASSIGNMENT Unsafe assignment to innerHTML Due to both security and performance concerns, this may not be set using dynamic values which have js/bundle.js 32079 4
not been adequately sanitized. This can lead to security issues or fairly serious performance
degradation.
UNSAFE_VAR_ASSIGNMENT Unsafe assignment to innerHTML Due to both security and performance concerns, this may not be set using dynamic values which have js/bundle.js 32086 5
not been adequately sanitized. This can lead to security issues or fairly serious performance
degradation.
Only a small issue with bootstrap fonts and webpack left 👍
Doing the submission process right now
Only warning s so everything should be ok 🍻
Hello,
Your add-on, webextensions History Browser 1.0.0, has been approved and is now available for download in our gallery at https://addons.mozilla.org/addon/webextensions-history-browser/
Reviewer: Mozilla
Comments: This version has been screened and approved for the public. Keep in mind that other reviewers may look into this version in the future and determine that it requires changes or should be taken down. In that case, you will be notified again with details and next steps.
Thank you!
🎉
So I tried to submit this, but I'm getting some linter errors even tho I started with linting.
They're linking to https://github.com/mozilla/addons-linter/ so I may have to set something like that in the project.
Here's what I tried for the release (started from scratch to make sure release was clean)
Then I uploaded
webextensions-history-browser.zip
.