Skip to content
This repository was archived by the owner on Aug 26, 2026. It is now read-only.

chore: gitignore the GitHub App private key temp file - #767

Merged
ExtraToast merged 1 commit into
mainfrom
chore/gitignore-github-app-key
Aug 26, 2026
Merged

chore: gitignore the GitHub App private key temp file#767
ExtraToast merged 1 commit into
mainfrom
chore/gitignore-github-app-key

Conversation

@ExtraToast

Copy link
Copy Markdown
Owner

What

app-pk.tmp holds the GitHub App private key, pasted out of the App settings page during the Flux source cutover. It was sitting untracked and unignored in the repository root — one git add . from being committed.

The credentials block in .gitignore already covers *.pem, *.key, *.p12 and *.pfx. The .tmp extension meant none of them matched.

Checked before deleting the local copy

Never exposed. No commit touches that path, and searching every object in every ref for the key body returns nothing:

commits touching that path: 0
objects containing the key's second line: 0

So no rotation is needed.

Not the only copy. The key lives in the githubAppPrivateKey entry of the flux-system Secret, which is what Flux authenticates to GitHub with. Confirmed by fingerprint rather than by reading either value:

flux-system/githubAppPrivateKey: 1679 bytes  sha256=1f7e17a8df8476f8...
app-pk.tmp:                      1679 bytes  sha256=1f7e17a8df8476f8...

Identical, so deleting the working copy loses nothing.

Note

That cluster Secret is now the only copy. A GitHub App private key cannot be re-downloaded, so if it is ever lost the recovery is to generate a new one in the App settings and update the Secret — not a problem, just worth knowing.

app-id.tmp is left in place and unignored deliberately: an App ID is not a secret.

app-pk.tmp holds the GitHub App private key, pasted out of the App settings page
during the Flux source cutover. It sat untracked and unignored in the repository
root, one `git add .` away from being committed.

The credentials block above it already covers *.pem, *.key, *.p12 and *.pfx, but
the file carries a .tmp extension so none of those patterns matched it.

History was checked before removing the local copy: no commit touches that path
and no object in any ref contains the key body, so the key was never exposed and
needs no rotation. The live copy is the githubAppPrivateKey entry in the
flux-system Secret, which is what Flux authenticates with -- verified by
comparing SHA-256 of both, 1679 bytes and identical.
@ExtraToast ExtraToast added the ci CI pipeline / Pipeline Complete label Aug 26, 2026
@ExtraToast ExtraToast self-assigned this Aug 26, 2026
@ExtraToast
ExtraToast merged commit 23a56a2 into main Aug 26, 2026
22 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ci CI pipeline / Pipeline Complete

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant