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

Ensure enable-submissions Disabling Correctly Allows Bypassing #22834

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

chrstinalin
Copy link
Contributor

@chrstinalin chrstinalin commented Nov 7, 2024

Fixes: mozilla/addons#15096

Description

Ensures users or categories of users are able to bypass enable-submissions disabling when allowed.

Testing

API calls to the following should be enabled/disabled accordingly when, for example, superusers or users on the flag enable-submissions is set.

  1. POST api/v5/addons/addon/
  2. PUT (New Addon) api/v5/addons/addon/<guid>/
  3. POST api/v5/addons/addon/<guid>/versions/
  4. POST api/v5/addons/addon/<guid>/previews/
  5. POST api/v5/addons/upload/
  6. PUT api/v4/addons/
  7. POST api/v4/addons/<guid>/versions/<version>
  8. PUT api/v3/addons/
  9. POST api/v3/addons/<guid>/versions/<version>

Checklist

  • Add #ISSUENUM at the top of your PR to an existing open issue in the mozilla/addons repository.
  • Successfully verified the change locally.
  • The change is covered by automated tests, or otherwise indicated why doing so is unnecessary/impossible.

@chrstinalin chrstinalin requested review from a team and KevinMind and removed request for a team November 7, 2024 18:12
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.

[Bug]: Users added in the flag or admins should be able to submit their own addons or new versions using api
3 participants