Skip to content

Fix mass-block of add-ons with at least one version already blocked#24569

Merged
diox merged 3 commits intomozilla:masterfrom
diox:mass-block-already-blocked-fix
Mar 5, 2026
Merged

Fix mass-block of add-ons with at least one version already blocked#24569
diox merged 3 commits intomozilla:masterfrom
diox:mass-block-already-blocked-fix

Conversation

@diox
Copy link
Copy Markdown
Member

@diox diox commented Mar 5, 2026

Mass-blocking (over GUID_FULL_LOAD_LIMIT add-ons) at once triggers the use of FakeBlockAddonVersion, but that doesn't have blockversion so it would fail when trying to determine the block type of an already blocked version.

Fixes mozilla/addons#16105

@diox diox force-pushed the mass-block-already-blocked-fix branch from 7a73676 to c52142e Compare March 5, 2026 14:01
Mass-blocking (over GUID_FULL_LOAD_LIMIT add-ons) at once triggers
the use of FakeBlockAddonVersion, but that doesn't have blockversion
so it would fail when trying to determine the block type of an
already blocked version.
@diox diox force-pushed the mass-block-already-blocked-fix branch from c52142e to 2703f02 Compare March 5, 2026 14:27
@diox diox requested a review from eviljeff March 5, 2026 14:27
@diox diox merged commit 5f4a0f2 into mozilla:master Mar 5, 2026
166 of 172 checks passed
diox added a commit that referenced this pull request Mar 10, 2026
…24569)

* Fix mass-block of add-ons with at least one version already blocked

Mass-blocking (over GUID_FULL_LOAD_LIMIT add-ons) at once triggers
the use of FakeBlockAddonVersion, but that doesn't have blockversion
so it would fail when trying to determine the block type of an
already blocked version.
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]: Mass-blocking add-ons with an already blocked version in the middle fails with 500

2 participants