Repository navigation
Conversation
Fixes GHSA-j38q-h3ph-jjpq (SQL injection in simple-protocol batches) and GHSA-95fj-3fxw-g62g (require_auth / channel_binding bypass, affecting v5.9.0 through v5.11.0). pgx is linked into step through github.com/smallstep/nosql.
This was referenced Nov 12, 2026
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.
Name of feature:
Just a dependency bump for a security fix. No behavior change.
Pain or issue this feature alleviates:
steppulls inpgxindirectly (viasmallstep/certificates→smallstep/nosql) and it's sitting at v5.9.2, which is affected by two advisories:require_auth/channel_bindingbypass (5.9.0 through 5.11.0)This bumps it to v5.12.0. The only other thing that moves is
puddle/v2(2.2.2 → 2.2.3). I skipped 5.13.0 on purpose since it changes how client certs get loaded - seemed better reviewed on its own. I'm guessing Dependabot missed this one because it's an indirect dependency.Why is this important to the project (if not answered above):
Bit of context: I've been chasing down the scanner findings in
smallstep/step-ca:0.30.2.step-cli:latestis the base for the step-ca and step-kms-plugin images, and it's also where theirstepbinary comes from. Xray pins 104 of the 231 findings in step-ca on thatstepbinary.Good news is master is already in great shape. With this bump, Trivy goes from 32 findings in the
stepbinary of the publishedstep-cli:0.31.0down to 1, and that last one is just thex/crypto/openpgpdeprecation notice (GO-2026-5932).stepdoesn't importopenpgp, and govulncheck says it's never called. The only fix is x/crypto v0.59.0, which drops DSA fromx/crypto/ssh-step ssh fingerprinton anssh-dsskey would stop working - so I left that for you and Dependabot.The big ask: if a CLI release could go out before the end-of-November step-ca release, that's what actually gets all of this into the step-ca image. Otherwise step-ca keeps shipping the September
stepbinary.Is there documentation on how to use this feature? If so, where?
N/A - nothing user-facing.
In what environments or workflows is this feature supported?
make build, and CodeQL. All green, and CodeQL comes back identical to master.docker/Dockerfileanddocker/Dockerfile.debian, then used them as the base for the step-ca and step-kms-plugin images in the companion PRs. Those all pass end-to-end checks: CA init, health, issuing certs,step kms, and a real Postgres backend.In what environments or workflows is this feature explicitly NOT supported (if any)?
None.
Supporting links/other PRs/issues: