Skip to content

Allow scanners to run asynchronously and send their results later#24447

Merged
diox merged 1 commit intomasterfrom
async-scanners
Feb 17, 2026
Merged

Allow scanners to run asynchronously and send their results later#24447
diox merged 1 commit intomasterfrom
async-scanners

Conversation

@willdurand
Copy link
Member

@willdurand willdurand commented Feb 10, 2026

Fixes mozilla/addons#16004


This PR lets scanners (webhooks) push their results async.

@desiorac

This comment was marked as outdated.

@willdurand willdurand force-pushed the async-scanners branch 5 times, most recently from e041f2e to de37ca6 Compare February 11, 2026 09:11
@willdurand willdurand marked this pull request as ready for review February 11, 2026 09:17
@willdurand willdurand requested a review from diox February 11, 2026 09:17
return Response(status=status.HTTP_404_NOT_FOUND)

# Verify the authenticated user is the service account for this webhook.
if request.user != scanner_result.webhook.service_account:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this was a class-based-view I would recommend doing this bit in a permission class (defining has_object_permission()) and just have that class in permission_classes but that doesn't work for api_view() ... It's probably fine.

@willdurand
Copy link
Member Author

(depends on #24462)

@willdurand
Copy link
Member Author

willdurand commented Feb 17, 2026

(rebase)

@diox diox merged commit dbc6bf3 into master Feb 17, 2026
90 of 92 checks passed
@diox diox deleted the async-scanners branch February 17, 2026 16:29
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.

[Task]: Implement async scanners

3 participants