Skip to content

More robust read of JSON returned by boundary CLI tool - #2125

Merged
rolandwalker merged 1 commit into
mainfrom
RW/boundary-robust-json-read
Aug 11, 2026
Merged

More robust read of JSON returned by boundary CLI tool#2125
rolandwalker merged 1 commit into
mainfrom
RW/boundary-robust-json-read

Conversation

@rolandwalker

Copy link
Copy Markdown
Contributor

Description

At the establishment of a tunnel

  • check for a status_code property in the emitted JSON and exit with its value if present
  • wrap access of credentials in a try block

It might be good to provide for overriding the user/pass with the usual values, in case Boundary is providing only a TCP connection.

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

@rolandwalker rolandwalker self-assigned this Aug 11, 2026
@rolandwalker
rolandwalker force-pushed the RW/boundary-robust-json-read branch from 882d075 to bf7a2e0 Compare August 11, 2026 08:57
At the establishment of a tunnel

 * check for a status_code property in the emitted JSON, and exit with
   its value if present
 * wrap access of credentials in a try block

It might be good to provide for overriding the user/pass with the
usual values, in case Boundary is providing only a TCP connection.
@rolandwalker
rolandwalker force-pushed the RW/boundary-robust-json-read branch from bf7a2e0 to dc07567 Compare August 11, 2026 09:01
@rolandwalker
rolandwalker merged commit 037f912 into main Aug 11, 2026
11 checks passed
@rolandwalker
rolandwalker deleted the RW/boundary-robust-json-read branch August 11, 2026 09:13
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.

1 participant