Skip to content

Bugzilla webhook authorization - #6786

Open
ayoubdiourin7 wants to merge 6 commits into
mozilla:masterfrom
ayoubdiourin7:bugzilla-webhook-authorization
Open

Bugzilla webhook authorization#6786
ayoubdiourin7 wants to merge 6 commits into
mozilla:masterfrom
ayoubdiourin7:bugzilla-webhook-authorization

Conversation

@ayoubdiourin7

Copy link
Copy Markdown
Contributor

Fixes #6755

@ayoubdiourin7
ayoubdiourin7 requested a review from a team as a code owner September 4, 2026 12:47
Comment thread docs/hackbot/triggers.md Outdated
Comment thread docs/hackbot/triggers.md Outdated
_REQUEST_TIMEOUT_SECONDS = 30


class BugzillaUserClient:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It should be BugzillaClient, similer to PhabricatorClient. We have an issue to resolve that in /libs: #6307

As a temporary solution, you move this logic to BugzillaAuthorizer, and add a TODO comment to mention #6459.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done in 465f2f2

Comment thread services/hackbot-api/app/config.py Outdated
Comment thread services/hackbot-api/app/bugzilla_webhook.py Outdated
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.

Bugzilla webhook should not respond to users without editbug permissions

2 participants