Skip to content

chore: better erroring when session expired/invalid#665

Merged
mcncl merged 2 commits intomainfrom
SUP-6292/better_auth_errors
Feb 25, 2026
Merged

chore: better erroring when session expired/invalid#665
mcncl merged 2 commits intomainfrom
SUP-6292/better_auth_errors

Conversation

@mcncl
Copy link
Contributor

@mcncl mcncl commented Feb 25, 2026

Description

Our erroring isn't clear on how the user might get past the expired session, so we're improving the hinting but also making it less jargon-y

Changes

  • allows for no org to be set; this is entirely acceptable for only view public pipelines
  • updates errors to better inform on the issue
  • adds hints at how to progress

Testing

  • Tests have run locally (with go test ./...)
  • Code is formatted (with go fmt ./...)

Our erroring isn't clear on how the user might get past the expired session, so we're improving the hinting but also making it less jargon-y
@mcncl mcncl requested a review from a team as a code owner February 25, 2026 01:16
@mcncl mcncl enabled auto-merge (squash) February 25, 2026 03:47
Copy link
Contributor

@omehegan omehegan left a comment

Choose a reason for hiding this comment

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

I'd suggest we put the commands and args in single quotes or backticks just to make sure they stand out from the rest of the message.

Edit: Oh, auto-merge. Valla con dios.

@mcncl mcncl merged commit fa634e1 into main Feb 25, 2026
1 check passed
@mcncl mcncl deleted the SUP-6292/better_auth_errors branch February 25, 2026 23:46
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.

2 participants