fix(version): pass TLS options in /status probe + mark <repo> as optional in help - #43
Merged
Merged
Conversation
…onal in help
Two small operator-pain reductions reported in the same session:
1. 'deltix version' showed "server version probe unavailable" on
self-signed TLS servers even though every data command worked. The
/status probe was a raw fetch() without buildFetchTlsOptions(...);
once the CA cert + SNI override are threaded through, the probe
succeeds just like the rest of the API.
2. The top-level deltix help still showed every <repo> as required
even though v0.7.14 made all data commands autodetect from cwd
when no arg is given. Marked them as [<repo>] so the help reflects
reality, added the discoverability note at the top ('When run from
a deltix init-ed working tree, [<repo>] becomes optional'), and
standardised the [--flag] patterns to [--flag=value|-f value] so
the -b / -n short flags landed in v0.7.13 show up too.
133 unit tests pass; lint clean.
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.
Two small UX fixes reported together: (1) 'deltix version' showed 'server probe unavailable' on self-signed TLS servers — the probe path used a raw fetch() without buildFetchTlsOptions(...). Fix: thread the same TLS options through. (2) The top-level help still showed every as required even though v0.7.14 made all data commands autodetect from cwd. Mark them as [] and add the discoverability note.
133 unit tests pass; lint clean.