Skip to content

Handle docker persistent login error and keeper get lookup#2121

Merged
amangalampalli-ks merged 2 commits into
fix/keeper-get-and-docker-login-errorfrom
fix/keeper-get-and-docker-login-error-int
Jun 8, 2026
Merged

Handle docker persistent login error and keeper get lookup#2121
amangalampalli-ks merged 2 commits into
fix/keeper-get-and-docker-login-errorfrom
fix/keeper-get-and-docker-login-error-int

Conversation

@amangalampalli-ks

Copy link
Copy Markdown
Contributor

Summary

Improves Commander behavior in headless/service environments when persistent login breaks, and adds defense-in-depth validation on get lookups.

Changes

  • Fail fast in non-interactive login when persistent login needs a follow-up step (password, 2FA, device approval, SSO)
  • Reject get lookup values containing shell chaining markers; allow normal title punctuation

@amangalampalli-ks amangalampalli-ks self-assigned this Jun 5, 2026
@amangalampalli-ks amangalampalli-ks changed the title Add error handling for persistent login break in docker and strengthe… Handle docker persistent login error and keeper get lookup Jun 5, 2026
@amangalampalli-ks
amangalampalli-ks marked this pull request as ready for review June 5, 2026 11:39
@amangalampalli-ks
amangalampalli-ks merged commit eea2308 into fix/keeper-get-and-docker-login-error Jun 8, 2026
4 checks passed
amangalampalli-ks added a commit that referenced this pull request Jun 8, 2026
* Add error handling for persistent login break in docker and strengthen keeper get lookup from shell

* Fix unit tests
craiglurey pushed a commit that referenced this pull request Jun 8, 2026
…2133)

* Add error handling for persistent login break in docker and strengthen keeper get lookup from shell

* Fix unit tests
jpkeepersecurity pushed a commit that referenced this pull request Jun 9, 2026
…2133)

* Add error handling for persistent login break in docker and strengthen keeper get lookup from shell

* Fix unit tests
sk-keeper pushed a commit that referenced this pull request Jun 13, 2026
…2133)

* Add error handling for persistent login break in docker and strengthen keeper get lookup from shell

* Fix unit tests
sk-keeper pushed a commit that referenced this pull request Jun 13, 2026
…2133)

* Add error handling for persistent login break in docker and strengthen keeper get lookup from shell

* Fix unit tests
sshrushanth-ks pushed a commit that referenced this pull request Jun 16, 2026
…2133)

* Add error handling for persistent login break in docker and strengthen keeper get lookup from shell

* Fix unit tests
@sk-keeper
sk-keeper deleted the fix/keeper-get-and-docker-login-error-int branch June 26, 2026 18:10
pvagare-ks pushed a commit that referenced this pull request Jul 16, 2026
…2133)

* Add error handling for persistent login break in docker and strengthen keeper get lookup from shell

* Fix unit tests
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