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

Unlisted submissions with a delay auto-approval flag are still auto-approved shortly #7128

Closed
AlexandraMoga opened this issue Nov 18, 2019 · 4 comments · Fixed by mozilla/addons-server#12904

Comments

@AlexandraMoga
Copy link

Follow up for #7047 and #7064

Describe the problem and steps to reproduce it:

  1. Log in to AMO -dev
  2. Submit a new unlisted extension that contains a matched rule configured with a Delay auto-approval action (24h or indefinite)
  3. Check the add-on status after a few minutes

What happened?

The add-on is approved

What did you expect to happen?

The add-on is still in an awaiting review state

Anything else we should know?

  • looking at the Flagged by scanners queue, the add-ons will appear in the list with the corresponding delay flags:
    image
  • both add-ons have a a scanner result with a corresponding matched rule
@eviljeff
Copy link
Member

I don't know if we're relying on these rules working yet. If we are its a P1, otherwise I dunno @diox @willdurand

@diox
Copy link
Member

diox commented Nov 19, 2019

We aren't yet, especially for unlisted, but I'll take a look.

@diox diox self-assigned this Nov 19, 2019
@diox diox added this to the 2019.11.21 milestone Nov 19, 2019
@diox
Copy link
Member

diox commented Nov 19, 2019

The auto_approval_delayed_until field was set properly on the AddonReviewerFlags instance, and the AutoApprovalSummary instance was created immediately after, but somehow when it ran check_has_auto_approval_disabled() it didn't see the delay.

I suspect somehow the AddonReviewerFlags instance had already been fetched and cached, as OneToOneField are, although I can't find what would have triggered that - it would have needed to be accessed first, but AFAIK that doesn't happen... I'll investigate.

@AlexandraMoga
Copy link
Author

Verified fixed on -dev.

Unlisted submissions that matched a rule with a delay auto-approval action will no longer be auto-approved shortly after submission. Those with a 24h delay will be auto-approved after the flag expires.

@KevinMind KevinMind transferred this issue from mozilla/addons-server May 4, 2024
@KevinMind KevinMind added repository:addons-server Issue relating to addons-server migration:2024 labels May 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants