Add NSF support for pam rotation script list/add/edit/delete#213
Add NSF support for pam rotation script list/add/edit/delete#213mtyagi-ks wants to merge 2 commits into
Conversation
| """Update an NSF typed record, including record-link adds/removes for file/script refs. | ||
|
|
||
| Mirrors classic ``record_management.update_record`` so PAM rotation scripts | ||
| (and other fileRef/script attachments) work on Keeper Drive records. |
There was a problem hiding this comment.
remove keeper drive comment and remove comment start, instead of mirrors we can right that it is in sync with update record flow for classic
|
|
||
|
|
||
| def update_nsf_typed_record( | ||
| vault: VaultOnline, |
There was a problem hiding this comment.
this isn't an existing method? also if it is a new implementation we can add batch support to it like batch record add
There was a problem hiding this comment.
it’s intentionally new to mirror classic update_record for NSF typed records + links; batch can be a follow-up since the current caller is single-record and classic update is also single-record.
There was a problem hiding this comment.
this method is NSF equivalent of classic update_record for TypedRecord + links
update_nsf_record (existing) is Dict/field update of NSF payload only — no record_links_add / remove
3 participants
Summary
pam rotation script(list/add/edit/delete) for pamUser/pamDirectory records in Nested Shared Folders.update_nsf_typed_recordso NSF updates can add/remove record links for script file attachments.script.fileRefin typed-record ref extraction so classic and NSF script updates keep file links in sync.Test plan
pam rotation script add --script <file> <nsf-pamUser-uid>uploads and attaches the scriptpam rotation script list <nsf-pamUser-uid>shows script UID/name/commandpam rotation script edit --script <uid|name> --script-command "..." <nsf-pamUser-uid>persists commandpam rotation script delete --script <uid|name> <nsf-pamUser-uid>removes the scriptpam-nsf-support-int