Skip to content

Misc fixes with PAM Tunnel Diagnose command - #2255

Open
lthievenaz-keeper wants to merge 2 commits into
Keeper-Security:releasefrom
lthievenaz-keeper:fix-pamTunnelDiagnose/fix-server+region+colorCoding
Open

Misc fixes with PAM Tunnel Diagnose command#2255
lthievenaz-keeper wants to merge 2 commits into
Keeper-Security:releasefrom
lthievenaz-keeper:fix-pamTunnelDiagnose/fix-server+region+colorCoding

Conversation

@lthievenaz-keeper

@lthievenaz-keeper lthievenaz-keeper commented Jul 31, 2026

Copy link
Copy Markdown
Contributor
  • 443 server URL tested is wrong for GOV (connect.govcloud.keepersecurity.us instead of connect.keepersecurity.us)
  • Wrong region printed for non-US servers (KEEPERSECURITY instead of EU, CA, JP, AU)
  • Websocket test is always passed even if response status is 400 or 500
  • WebRTC sample color-coding is incomplete
  • Command returns an int

Before

image

After

image

- 443 server test does not target the correct server for GOV
- region does not display correctly for non-US servers
- Websocket test returns green even for 400/500 errors
- Green color coding is partially applied on webRTC samples
PAM Tunnel Diagnose command returns an int in the CLI, which no other command does, removing int return statements.
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