Prepare postgres-barman to be public - #2
Merged
Merged
Conversation
Prepares the repo to go public. No credentials were ever committed, so there is no history to rewrite -- and rewriting it would break clones and invalidate the 18-20260712 tag that consumers pin in production. The image was built generic on purpose, so the audit is short: only the prose named the internal consumers. The README's diagram, its "every MetsaApp database" line and its Kamal example (which hardcoded a real user, bucket, server name and object-storage endpoint) become placeholders. The Dockerfile's header comment named the same three consumers and gets the same treatment -- comment-only, so the digest, the build assertions and the published image are unchanged. LICENSE is MIT. SECURITY.md points reports at private advisories and is honest that most of the attack surface here is upstream Postgres and Barman.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #1 (the code half of it).
The image was built generic on purpose -- "nothing in it knows which database it will hold" -- and that held up, so the audit is short. Only the prose named the internal consumers.
What changed
oresultuser/db,s3://o-result-backups/api, theo-result-productionserver name and the Hetznerfsn1endpoint become obvious placeholders.Everything else stays. The CloudNativePG rejection, the
PG_VERSION/initdbtrap and the digest-pinning rationale are generic, and are more useful in public than in private.No history rewrite. There are no credentials in any commit, and force-pushing would break clones and invalidate the
18-20260712tag pinned in production.Checks
grep -riE 'o-result|oresult|metsa api|zitadel|fsn1|your-objectstorage'over the tree is clean. I could not rundocker buildlocally (no Docker in this WSL distro) -- the PR build here is the real check.Still to do, outside this PR
mainFROMthis image; a public pgctl pointing at a private base is worse than useless.