Skip to content

fix(cli): Ensure @authenticated routes require auth with external providers#266

Merged
dnys1 merged 1 commit intomainfrom
03-18-fix_cli_ensure_authenticated_routes_require_auth_with_external_providers
Mar 19, 2025
Merged

fix(cli): Ensure @authenticated routes require auth with external providers#266
dnys1 merged 1 commit intomainfrom
03-18-fix_cli_ensure_authenticated_routes_require_auth_with_external_providers

Conversation

@dnys1
Copy link
Member

@dnys1 dnys1 commented Mar 19, 2025

This ensures that APIs throw the appropriate error and return the correct status code when a user is not authenticated.

Copy link
Member Author

dnys1 commented Mar 19, 2025

Copy link
Member Author

dnys1 commented Mar 19, 2025

Merge activity

  • Mar 19, 8:05 AM PDT: A user started a stack merge that includes this pull request via Graphite.
  • Mar 19, 8:10 AM PDT: Graphite rebased this pull request as part of a merge.
  • Mar 19, 8:11 AM PDT: A user merged this pull request with Graphite.

@dnys1 dnys1 changed the base branch from 03-18-fix_cli_websocket_error_condition to graphite-base/266 March 19, 2025 15:06
@dnys1 dnys1 changed the base branch from graphite-base/266 to main March 19, 2025 15:08
…roviders

This ensures that APIs throw the appropriate error and return the correct status code when a user is not authenticated.
@dnys1 dnys1 force-pushed the 03-18-fix_cli_ensure_authenticated_routes_require_auth_with_external_providers branch from 8b7496f to e0fa5f0 Compare March 19, 2025 15:09
@dnys1 dnys1 merged commit ed5361b into main Mar 19, 2025
4 checks passed
@dnys1 dnys1 deleted the 03-18-fix_cli_ensure_authenticated_routes_require_auth_with_external_providers branch March 19, 2025 15:11
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.

1 participant