Skip to content

fix(domain): avoid duplicating railway-verify prefix#862

Open
onthebed wants to merge 1 commit into
railwayapp:masterfrom
onthebed:fix-domain-verification-txt-format
Open

fix(domain): avoid duplicating railway-verify prefix#862
onthebed wants to merge 1 commit into
railwayapp:masterfrom
onthebed:fix-domain-verification-txt-format

Conversation

@onthebed
Copy link
Copy Markdown

Summary

  • add a small formatter that only prepends railway-verify= when the token is bare
  • reuse that formatter for railway domain DNS output and the matching MCP verification output
  • add regression tests covering both prefixed and bare verification tokens

Closes #851

Testing

  • cargo fmt --all -- --check
  • cargo check
  • cargo clippy --all-targets --all-features
  • cargo test

@onthebed
Copy link
Copy Markdown
Author

The only failing check on this PR is , which requires one of the release labels (, , , or ). I tried to add from my side, but GitHub permissions for this fork do not allow adding labels. If a maintainer can apply the appropriate release label, the PR should be unblocked on that check.

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.

CLI: railway domain outputs malformed TXT value (duplicated railway-verify= prefix) — silently breaks cert provisioning

1 participant