Skip to content

Fix/MM-11 download membership timeout error - #2435

Merged
sk-keeper merged 5 commits into
releasefrom
fix/MM-11-download-membership-timeout-error
Oct 21, 2026
Merged

sk-keeper merged 5 commits into
releasefrom
fix/MM-11-download-membership-timeout-error

Conversation

@hpadalkar-ks

@hpadalkar-ks hpadalkar-ks commented Oct 12, 2026 •

Copy link
Copy Markdown

MM-11
Issue - Link
Client was trying to run download-membership command and was getting read timeout issue.

Findings -

  • They had 106 Users and around 190 safes in the CyberARK.
  • When fetching CyberARK Safes, API was failing with ReadTimeout Error.
    Request to /PasswordVault/API/Safes failed: HTTPSConnectionPool, port=443): Read timed out. (read timeout=10)

Fix -

  • Gave two arguments to download-membership command - timeout and api-page-size-limit
  • By doing this, all the users have the flexibility to change the page size of API and increase the timeout. If this error occurs in future.

Example -

# Download Membership with custom timeout and Page Size Limit  

download-membership --source=cyberark --timeout=20 --api-page-size-limti=100 import_file_name.json

erinlewis-keeper and others added 4 commits October 7, 2026 19:12
* MM-11 download membership timeout error

* MM-11 download membership timeout error

* MM-11 Fixed the args sequence in the methods

* MM-11 Fixed the args sequence in the methods

* MM-11 refactored and optimized the code for easy debugging
…into fix/MM-11-download-membership-timeout-error
@hpadalkar-ks hpadalkar-ks self-assigned this Oct 12, 2026
…ip (#2436)

* MM-11 written test cases for the functionality

* MM-11 Worked on test case review

* MM-11 fixed linting issues
@sk-keeper
sk-keeper merged commit cc78341 into release Oct 21, 2026
4 checks passed
@sk-keeper
sk-keeper deleted the fix/MM-11-download-membership-timeout-error branch October 21, 2026 05:14
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.

4 participants