Repository navigation
chore: release prep — version injection, CHANGELOG (closes #49) - #72
Merged
Merged
Conversation
Resolves #49 by completing the remaining release preparation items. What changed: - version/commit/date are now vars (not const) so goreleaser's ldflags can inject them at build time. Plain 'go build' shows 'hoot dev (commit none, built unknown)'; tagged releases show the real version, commit hash, and build date. - 'hoot -version' output now includes commit and date. - Added CHANGELOG.md covering all fixes from PRs #65-#71. - Updated TestVersion to match the new output format. What was already done (checked off in the issue): - .goreleaser.yml with multi-platform builds + ldflags - .github/workflows/release.yml (tag-triggered) - .github/workflows/ci.yml (tests + cross-build matrix) - LICENSE (MIT) - README.md - .gitignore (binary exclusions) - Config dir uses 'hoot' (not 'nostr-cli') - 9 test files across 3 packages Closes #49
oth-body
pushed a commit
that referenced
this pull request
Sep 15, 2026
Hoot's NIP-46 sign-in flow uses github.com/nbd-wtf/go-nostr's
nostr.RelayConnect, which delegates to net/http and therefore
honors HTTPS_PROXY / HTTP_PROXY / ALL_PROXY / SOCKS_PROXY env
vars inherited from the parent shell. If the user runs hoot
inside torsocks, a launcher that exports HTTPS_PROXY, or a shell
that already has Tor configured for monerod / curl, the wss://
dials to relay.damus.io / relay.nostr.band / nostr.wine (or
whatever the user picked) get routed through 127.0.0.1:9050.
When nothing is listening there — which is the typical case on
a workstation without an active Tor — every dial fails with
'dial tcp 127.0.0.1:9050: connect: connection refused', which
bubbles up as 'failed to connect to port 9050' in the TUI and
gives the user no actionable signal.
Fix it two ways:
- nip46.ConnectRelays now refuses to dial at all when any
*_PROXY env var is set, returning a clear error that names
the offending env var and the proxy host:port. The user can
either unset it (or run hoot outside torsocks). This matches
what every other Nostr TUI does — nak, lume, etc. — none
of them proxy wss:// connections by default.
- Up-front validation of every configured relay URL (rejecting
blanks and non-ws:// / non-wss:// schemes) so a misconfigured
relays.txt surfaces a clear error instead of a confusing
websocket one.
Anchored against master after PR #69 / #70 / #72 restructured
the NIP-46 dial path (RelayURL string -> RelayURLs []string,
WaitForConnection -> ConnectRelays + CheckConnection). Tests
updated accordingly: ConnectRelays is the new dial entry point,
and validateRelayURLs is exercised with single-bad-URL and
mixed-good-and-bad inputs.
Tests in nip46/nip46_test.go pin:
- the proxy env var detection order (HTTPS_PROXY beats
HTTP_PROXY beats ALL_PROXY; socks fallbacks come last)
- that malformed or empty proxy values fall through to the
next valid var (matching Go's stdlib httpproxy behaviour)
- that ConnectRelays surfaces the env var name and port on
the user-visible error path (this is the bug's regression
guard)
- that blank and non-wss relay URLs are rejected up front
instead of producing a confusing websocket error later
- that Session.Close is idempotent when no dial was attempted
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
1 participant
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.
Completes the remaining items from #49 (release preparation checklist).
Changes:
version,commit,dateare nowvar(notconst) so goreleaser's ldflags inject them at build timehoot -versionshows commit hash and build dateCHANGELOG.mdcovering all fixes from PRs fix(tui): render QR code when WindowSizeMsg arrives after initQR #65-fix: address 11 bugs found in codebase review #71Already done (from prior work):
.goreleaser.yml, release.yml, ci.yml, LICENSE, README.md, .gitignore, config dir, tests
Closes #49