Skip to content

fix(repositories): Silence gitlab errors when performing auto-sync#114647

Merged
wedamija merged 2 commits intomasterfrom
danf/sync-repos-gitlab-override
May 4, 2026
Merged

fix(repositories): Silence gitlab errors when performing auto-sync#114647
wedamija merged 2 commits intomasterfrom
danf/sync-repos-gitlab-override

Conversation

@wedamija
Copy link
Copy Markdown
Member

@wedamija wedamija commented May 1, 2026

This more explicitly handles some gitlab specific errors and allows us to mute them.

Fixes SENTRY-5N0Z, SENTRY-5NAS

This more explicitly handles some gitlab specific errors and allows us to mute them.

Fixes SENTRY-5N0Z
@wedamija wedamija requested review from a team as code owners May 1, 2026 22:08
@wedamija wedamija requested a review from a team May 1, 2026 22:08
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 1, 2026
@wedamija wedamija changed the title fix(repositories): Silence github errors when performing auto-sync fix(repositories): Silence gitlab errors when performing auto-sync May 1, 2026
Comment on lines +177 to +178
# Self-hosted GitLab instances sometimes return HTML login/captcha
# pages instead of JSON. The response parser raises ValueError.
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.

Useful comments 👍

@wedamija wedamija merged commit 7b18db1 into master May 4, 2026
59 checks passed
@wedamija wedamija deleted the danf/sync-repos-gitlab-override branch May 4, 2026 17:52
cleptric pushed a commit that referenced this pull request May 5, 2026
…114647)

This more explicitly handles some gitlab specific errors and allows us
to mute them.

Fixes SENTRY-5N0Z, SENTRY-5NAS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants