Conversation
* Support passing stringified JSON as filename PAM Project Extend command requires physical file with sensitive credential data. This commit adds support for passing stringified JSON data in the filename directly, rather than a filepath. * Remove debug statement
* Add native JSON validation for BaseFileImporter BaseFileImporter is a process of the import command which only supports a file. Added support for stringified JSON. * Added support for stringified JSON import The JSON import supports: - JSON filename - ZIP filename Added support for stringified JSON * Remove redundant path validation in ApplyMembershipCommand Removed path validation from ApplyMembershipCommand. This is required for the ApplyMembershipCommand to support stringified JSON data instead of a physical JSON file. This validation is redundant as it is done inside do_import. It is not performed in other similar commands in this file.
- Added selected nodes in deletion alert - Added logging for all deletion steps - Added automator deletion - If user can list gateways, and there are gateways on nodes, list gateways that need to be moved and how - If node_delete fails, clarify that provisioning methods and gateways are not cleared
…ttle storm Add thread-safe caching with 60s TTL to get_all_gateways() which calls keeperapp's pam/get_controllers endpoint. Under concurrent load, multiple threads sharing the same cached result instead of each making independent API calls. Prevents exceeding the server-side rate limit (10 req/10s) and the resulting cascading throttle loop. Cache is invalidated on gateway removal to prevent stale data.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.