Skip to content

Treat 503 as an expected authentication response#743

Merged
Page- merged 1 commit into
masterfrom
expected-503
Jul 24, 2026
Merged

Treat 503 as an expected authentication response#743
Page- merged 1 commit into
masterfrom
expected-503

Conversation

@Page-

@Page- Page- commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

This avoids spamming error reports on a 503 and instead uses the auth failure delay mechanism which will reduce the load we create on the API and also on openvpn from reconnect attempts and help reduce subsequent 503s and other slowness

Change-type: patch

Change-type: major|minor|patch
Connects-to: #
See:
Depends-on:


Contributor checklist
  • Build output has been rebuilt and tested
  • Introduces security considerations
  • Tests are included
  • Documentation is added or changed
  • Affects the development, build or deployment processes of the component
Reviewer Guidelines
  • When submitting a review, please pick:
    • 'Approve' if this change would be acceptable in the codebase (even if there are minor or cosmetic tweaks that could be improved).
    • 'Request Changes' if this change would not be acceptable in our codebase (e.g. bugs, changes that will make development harder in future, security/performance issues, etc).
    • 'Comment' if you don't feel you have enough information to decide either way (e.g. if you have major questions, or you don't understand the context of the change sufficiently to fully review yourself, but want to make a comment)

@Page-
Page- requested a review from a team July 24, 2026 16:21
@Page-
Page- enabled auto-merge July 24, 2026 16:29
otaviojacobi
otaviojacobi previously approved these changes Jul 24, 2026
@Page-
Page- marked this pull request as draft July 24, 2026 16:37
auto-merge was automatically disabled July 24, 2026 16:37

Pull request was converted to draft

@Page-
Page- requested review from a team and otaviojacobi July 24, 2026 16:42
@Page-
Page- marked this pull request as ready for review July 24, 2026 16:42
@Page-
Page- enabled auto-merge July 24, 2026 16:42
This avoids spamming error reports on a 503 and instead uses the auth
failure delay mechanism which will reduce the load we create on the API
and also on openvpn from reconnect attempts and help reduce subsequent
503s and other slowness

Change-type: patch
@Page-
Page- merged commit e699d88 into master Jul 24, 2026
53 checks passed
@Page-
Page- deleted the expected-503 branch July 24, 2026 16:57
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.

3 participants