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

[Bug]: When a stash and a filter are uploaded together, there be dragons... #15200

Closed
1 task done
KevinMind opened this issue Nov 25, 2024 · 1 comment
Closed
1 task done
Labels
needs:info repository:addons-server Issue relating to addons-server

Comments

@KevinMind
Copy link
Contributor

KevinMind commented Nov 25, 2024

What happened?

Our cron job to upload block list records uses a single generation time to stamp all uploaded records. This means that a stash and a filter can share a time stamp.

This is a problem because FX is checking first if an addon is in a stash and then if it is not and if the latest filter is more recent than the stash, we check if it is in a filter.

If the filter has the same time as the stash, it will never be checked and any addons included in the filter will not be blocked/softblocked as they should.

What did you expect to happen?

If we upload a stash, we should set it's creation time to 1 millisecond earlier than the filter.

We should also make sure that change doesn't fundamentally break anything on AMO side.

Is there an existing issue for this?

  • I have searched the existing issues

┆Issue is synchronized with this Jira Task

@willdurand
Copy link
Member

With the implementation for #15014, this doesn't seem possible at the moment

@willdurand willdurand closed this as not planned Won't fix, can't repro, duplicate, stale Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs:info repository:addons-server Issue relating to addons-server
Projects
None yet
Development

No branches or pull requests

2 participants