Skip to content

fix: scope token insteadOf rewrites to the famedly org#77

Merged
krille-chan merged 1 commit into
mainfrom
fix/scope-token-rewrites-to-famedly
Jun 10, 2026
Merged

fix: scope token insteadOf rewrites to the famedly org#77
krille-chan merged 1 commit into
mainfrom
fix/scope-token-rewrites-to-famedly

Conversation

@nikzen

@nikzen nikzen commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The token-based insteadOf rewrites in dart-prepare currently rewrite all github.com URLs, injecting the token into clones of public repositories as well.
  • Xcode's Swift Package Manager cannot handle the rewritten URLs with embedded credentials and hangs indefinitely during dependency resolution (Firebase, Sentry, SDWebImage, …) — observed on build_ios in famedly/app#7173, where the job stalled for 28+ minutes after Fetching from https://github.com/google/GoogleAppMeasurement.git... while an identical build without the token path finishes in ~13 minutes.
  • Fix: scope the rewrites to github.com/famedly/. Private famedly dependencies keep working via the token; public clones (incl. all SPM packages) stay untouched. As a side effect the token is no longer attached to every GitHub clone.

Rewriting all of github.com injects the token into clones of public
repositories as well. Xcode's Swift Package Manager cannot handle the
rewritten URLs with embedded credentials and hangs indefinitely during
dependency resolution (Firebase, Sentry, etc.) on iOS builds.

Scoping the rewrites to github.com/famedly/ keeps authentication for
the private dependencies while leaving public clones untouched.
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