Skip to content

Preserve explicit cfctl token lane#9

Merged
rogu3bear merged 1 commit into
mainfrom
repair/cfctl-lane-env-precedence
May 12, 2026
Merged

Preserve explicit cfctl token lane#9
rogu3bear merged 1 commit into
mainfrom
repair/cfctl-lane-env-precedence

Conversation

@rogu3bear

Copy link
Copy Markdown
Owner

Summary

  • preserve caller-provided CF_TOKEN_LANE after shared/repo env files are sourced
  • add a static-contract regression proving command-time global lane selection beats env-file defaults

Proof

  • red first: ./scripts/verify_static_contract.sh failed with 'explicit CF_TOKEN_LANE was not preserved over env files'
  • green after fix: ./scripts/verify_static_contract.sh
  • live readback: CF_TOKEN_LANE=global ./cfctl list worker.script returned auth.lane=global and included jkca-web-book
  • live readback: CF_TOKEN_LANE=global ./cfctl list worker.route --zone jkca.me returned auth.lane=global and jkca.me routes to jkca-web-edge-router

Notes

  • no Cloudflare mutation, deploy, or route change was performed

@kilo-code-bot

kilo-code-bot Bot commented May 12, 2026

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • scripts/lib/cloudflare.sh
  • scripts/verify_static_contract.sh

Reviewed by grok-code-fast-1:optimized:free · 82,588 tokens

@rogu3bear rogu3bear merged commit 74e24c3 into main May 12, 2026
3 checks passed
@rogu3bear rogu3bear deleted the repair/cfctl-lane-env-precedence branch May 12, 2026 13:41
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