Skip to content

fix(workflows): add Nix access-tokens for private GitHub flake inputs#10

Open
nikzen wants to merge 2 commits into
mainfrom
fix/nix-access-tokens-for-private-flake-inputs
Open

fix(workflows): add Nix access-tokens for private GitHub flake inputs#10
nikzen wants to merge 2 commits into
mainfrom
fix/nix-access-tokens-for-private-flake-inputs

Conversation

@nikzen

@nikzen nikzen commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • mkNixGitAuthStep only configured git URL rewriting (git config --system url...insteadOf), which does not affect Nix's tarball fetcher
  • Nix fetches GitHub flake inputs as tarballs via the GitHub API (api.github.com/repos/.../tarball/...), not via git — so private repos returned HTTP 404 despite the ENGINEERING_STANDARDS_READ token being available
  • This adds access-tokens = github.com=<token> to /etc/nix/nix.conf and restarts the Nix daemon so it can authenticate tarball downloads from private repositories

Test plan

  • Verify a repo using engineering-standards as a private flake input can successfully run nix flake check in CI
  • Verify the git URL rewriting still works for git-based fetches

Made with Cursor

nikzen added 2 commits April 10, 2026 20:10
mkNixGitAuthStep only configured git URL rewriting, which does not
affect Nix's tarball fetcher. Nix fetches GitHub flake inputs as
tarballs via the GitHub API, not via git, so private repos returned
HTTP 404 despite the token being available.

Append access-tokens to /etc/nix/nix.conf and restart the daemon so
Nix can authenticate tarball downloads from private repositories.

Made-with: Cursor
The shared deny.toml template contains exceptions for all Famedly AGPL
crates, but not every project depends on all of them. cargo-deny fails
on unmatched exceptions by default.

Add unused-license-exception = "allow" so projects using this template
don't fail when they only depend on a subset of Famedly crates.

Also add famedly-operator to the AGPL exception list.

Made-with: Cursor
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