Skip to content

Add --keeper-db-proxy to pam tunnel edit and --launch-user to pam connection edit#1809

Merged
tbjones-ks merged 1 commit intoreleasefrom
45-add-keeper-db-proxy-option
Feb 12, 2026
Merged

Add --keeper-db-proxy to pam tunnel edit and --launch-user to pam connection edit#1809
tbjones-ks merged 1 commit intoreleasefrom
45-add-keeper-db-proxy-option

Conversation

@tbjones-ks
Copy link
Contributor

Summary

  • Move --keeper-db-proxy (-kdbp) option from pam connection edit to pam tunnel edit
  • Add --launch-user (-lu) option to pam connection edit for assigning launch credentials
  • Add launch credential validation when enabling DB proxy

Jira / GitHub Issue

keeperdb-proxy#45

Changes

  • pam tunnel edit: Added --keeper-db-proxy (on/off/default) to configure allowKeeperDBProxy PAM setting on pamDatabase records
  • pam tunnel edit: When enabling (on), validates that launch credentials are assigned to the record
  • pam connection edit: Added --launch-user (-lu) to assign a pamUser record as the launch credential on a PAM resource via DAG
  • pam connection edit: Removed --keeper-db-proxy (moved to tunnel edit)
  • TunnelDAG: Added check_if_resource_has_launch_credential() method
  • TunnelDAG: Added is_launch_credential parameter to link_user() and link_user_to_resource()

Breaking Changes

  • --keeper-db-proxy is no longer available on pam connection edit. Use pam tunnel edit instead.

Testing Performed

  • Syntax validation passes
  • Argument parsing verified for all new flags
  • Invalid choices correctly rejected
  • Manual testing with live environment (pending)

Usage

# Assign launch credentials to a pamDatabase record
pam connection edit <record> --launch-user <pamUser_record>

# Enable DB proxy (requires launch credentials)
pam tunnel edit <record> --keeper-db-proxy=on

# Disable DB proxy
pam tunnel edit <record> --keeper-db-proxy=off

# Reset to default
pam tunnel edit <record> --keeper-db-proxy=default

@tbjones-ks tbjones-ks self-assigned this Feb 12, 2026
@tbjones-ks tbjones-ks requested a review from maksimu February 12, 2026 00:41
@craiglurey craiglurey changed the base branch from master to release February 12, 2026 05:54
Change List:
   - Add --keeper-db-proxy (-kdbp) to pam tunnel edit for pamDatabase records
   - Add launch credential check when enabling DB proxy (--keeper-db-proxy=on)
   - Add --launch-user (-lu) flag to pam connection edit for setting launch credentials
   - Add check_if_resource_has_launch_credential() to TunnelDAG
   - Add is_launch_credential support to link_user/link_user_to_resource in TunnelDAG
   - Validate --launch-user record is pamUser type
@tbjones-ks tbjones-ks force-pushed the 45-add-keeper-db-proxy-option branch from 0bf7822 to 785b537 Compare February 12, 2026 17:13
@tbjones-ks tbjones-ks merged commit 8b92f53 into release Feb 12, 2026
4 checks passed
@tbjones-ks tbjones-ks deleted the 45-add-keeper-db-proxy-option branch February 12, 2026 17:15
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