Skip to content

[codex] fix railway token auth preflight#950

Merged
m-abdelwahab merged 1 commit into
masterfrom
mahmoud/fix-railway-token-up-auth
Jun 5, 2026
Merged

[codex] fix railway token auth preflight#950
m-abdelwahab merged 1 commit into
masterfrom
mahmoud/fix-railway-token-up-auth

Conversation

@m-abdelwahab
Copy link
Copy Markdown
Collaborator

Summary

  • Treat RAILWAY_TOKEN as valid credentials for the railway up auth preflight.
  • Keep user/workspace-token behavior intact by routing the preflight through a new Configs::has_auth_credentials() helper.
  • Add focused regression coverage for project-token credentials.

Root cause

The newer railway up first-run auth flow checked get_railway_auth_token() to decide whether to fail fast or start interactive login. That method intentionally excludes RAILWAY_TOKEN, so CI deployments using project tokens exited with Not signed in before the existing project-token deploy path could run.

Fixes #947.

Validation

  • cargo test

@m-abdelwahab m-abdelwahab marked this pull request as ready for review June 5, 2026 19:53
@m-abdelwahab m-abdelwahab added the release/patch Author patch release label Jun 5, 2026
@m-abdelwahab m-abdelwahab merged commit 11d7b04 into master Jun 5, 2026
6 of 7 checks passed
@m-abdelwahab m-abdelwahab deleted the mahmoud/fix-railway-token-up-auth branch June 5, 2026 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release/patch Author patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

railway/cli:5.0.0 and onwards broke RAILWAY_TOKEN deployments

1 participant