Skip to content

feat: desktop bridge login via Keeper Desktop (KC-1300)#2143

Draft
aaunario-keeper wants to merge 7 commits into
releasefrom
KC-1300
Draft

feat: desktop bridge login via Keeper Desktop (KC-1300)#2143
aaunario-keeper wants to merge 7 commits into
releasefrom
KC-1300

Conversation

@aaunario-keeper

Copy link
Copy Markdown
Contributor
  • Add --via-desktop flag to top-level CLI and login command
  • Auto-suppress startup login when first queued command is login --via-desktop
  • Implement exchange_vault_token bridge relay + direct KA login_from_existing_session_token
  • Auto-enroll device on fresh install (no config.json required)
  • Register DEDK after bridge login for future persistent logins
  • 12 unit tests covering bridge exchange, KA transport, auto-enrollment, error paths

idimov-keeper and others added 7 commits June 6, 2026 21:04
* Fix kcm-import --output file permissions on Windows via utils.set_file_permissions

* Fix flaky set_record_rotation body test: decrypt round-trip instead of first-byte check
…2133)

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

* Fix unit tests
- Add --via-desktop flag to top-level CLI and login command
- Auto-suppress startup login when first queued command is login --via-desktop
- Implement exchange_vault_token bridge relay + direct KA login_from_existing_session_token
- Auto-enroll device on fresh install (no config.json required)
- Register DEDK after bridge login for future persistent logins
- 12 unit tests covering bridge exchange, KA transport, auto-enrollment, error paths
--new-login (force fresh auth) and --via-desktop (auth method) are
orthogonal. Removed mutual exclusion from login_parser, __main__.py,
and LoginCommand.execute. Also removes top_level_new_login from params
since it only existed to enforce the now-removed restriction.
@sk-keeper sk-keeper force-pushed the release branch 2 times, most recently from b11acff to c6ec45e Compare June 13, 2026 02:41
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.

5 participants