Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Illegal values in optional_permissions should be accepted #1788

Closed
piroor opened this issue Jan 25, 2018 · 2 comments
Closed

Illegal values in optional_permissions should be accepted #1788

piroor opened this issue Jan 25, 2018 · 2 comments

Comments

@piroor
Copy link

piroor commented Jan 25, 2018

Describe the problem and steps to reproduce it:

Firefox now accepts illegal values in optional_permissions (after the Bug 1350759: Warn but accept illegal values in optional_permissions) but the linter still blocks any unknown permission names. Thus I cannot release my addon including new permission tabHide in optional permissions which is implemented at Firefox 59.

Steps to reproduce:

  1. Create an addon including tabHide in its optional_permissions.
  2. Upload it to AMO.

What happened?

I got 4 errors and couldn't release the version:

"/optional_permissions/2" should be equal to one of the allowed values

エラー: Your JSON file could not be parsed.
"/optional_permissions/2" should match some schema in anyOf

エラー: Your JSON file could not be parsed.
"/optional_permissions/2" should match pattern "^(https?|wss?|file|ftp|\*)://(\*|\*\.[^*/]+|[^*/]+)/.*$"

エラー: Your JSON file could not be parsed.
"/optional_permissions/2" should match pattern "^file:///.*$"

エラー: Your JSON file could not be parsed.

What did you expect to happen?

A new permission tabHide is warned but not reported as an error.

Anything else we should know?

(Please include screenshots and any relevant files.)

This is the file I uploaded.
treestyletab-we.xpi.zip

piroor added a commit to piroor/treestyletab that referenced this issue Jan 26, 2018
Due to mozilla/addons-linter#1788 we cannot release a version with "tabHide" permission.
@muffinresearch
Copy link
Contributor

Is this still reproducible for optional permissions?

@stale
Copy link

stale bot commented Dec 11, 2019

This issue has been automatically marked as stale because it has not had recent activity. If you think this bug should stay open, please comment on the issue with further details. Thank you for your contributions.

@stale stale bot added the state:stale label Dec 11, 2019
@stale stale bot closed this as completed Dec 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants