Skip to content

*: add multiple password support for servers and relays - #24

Merged
bruce-y merged 3 commits into
mainfrom
bruce-y/multiple-passwords-support
Oct 20, 2026
Merged

bruce-y merged 3 commits into
mainfrom
bruce-y/multiple-passwords-support

Conversation

@bruce-y

@bruce-y bruce-y commented Oct 20, 2026 •

Copy link
Copy Markdown

Servers and relays can now accept multiple passwords through comma-separated VPROX_PASSWORDS, alongside the existing literal VPROX_PASSWORD, enabling password rotation and separate client credentials.
Clients continue using one password, and the relay generates its authentication reply with the matching password without changing the wire protocol.
Adds a precomputed hash map for HTTP authentication, empty-password validation, CLI help and rotation documentation, plus tests for parsing, legacy compatibility, both authentication paths, and replay rejection.

Validation passed: go test -race ./... and CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build.


View with [code]smith Show before/after
Need help on this PR? Tag @codesmith-bot with what you need.


View with [code]smith View full Proof
Need help on this PR? Tag @codesmith-bot with what you need. (Staging)

@blacksmith-staging

blacksmith-staging Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Checked Relay accepts every listed password

Before, only the VPROX_PASSWORD client got in. After, all three configured passwords open a session with a server proof that verifies for that client's password, and a wrong password is still refused.

Merge-base

Relay hello before

This PR 6e1cbb91

Relay hello after

Also checked · 2 more claims, all matched observations

Checked Server /connect accepts every listed password
Checked Password list parsing edges

Proof on 6e1cbb91

View full Proof report

@bruce-y bruce-y changed the title Add multiple password support for servers and relays *: add multiple password support for servers and relays Oct 20, 2026
@bruce-y
bruce-y merged commit c618c01 into main Oct 20, 2026
8 checks passed
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