Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug fix/XSUP 41283/CyberArk EPM improve the error message #36570

Conversation

MosheEichler
Copy link
Contributor

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: link to the issue

Description

A few sentences describing the overall goals of the pull request's commits.

Must have

  • Tests
  • Documentation

@MosheEichler MosheEichler changed the title Bug fix/XSUP 41283/CyberArkIdentity improve the error message Bug fix/XSUP 41283/CyberArk Identity improve the error message Sep 30, 2024
@MosheEichler MosheEichler self-assigned this Sep 30, 2024
Copy link

github-actions bot commented Sep 30, 2024

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/CyberArkEPM/Integrations/CyberArkEPMEventCollector
   CyberArkEPMEventCollector.py133794%40, 115, 245–247, 353–354
TOTAL133794% 

Tests Skipped Failures Errors Time
10 0 💤 0 ❌ 0 🔥 1.809s ⏱️

@@ -65,7 +65,7 @@ def get_session_token(self) -> str: # pragma: no cover
demisto.debug(f"result is: {result}")
if result.get("status", "") != "SUCCESS":
raise DemistoException(f"Retrieving Okta session token returned status: {result.get('status')},"
f" check your okta credentials please.")
f" please check your okta credentials and make sure the user is not blocked by a role.")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
f" please check your okta credentials and make sure the user is not blocked by a role.")
f" Check your Okta credentials and make sure the user is not blocked by a role.")

@ShirleyDenkberg
Copy link
Contributor

@thefrieddan1 Doc review completed.

@MosheEichler MosheEichler changed the title Bug fix/XSUP 41283/CyberArk Identity improve the error message Bug fix/XSUP 41283/CyberArk EPM improve the error message Oct 1, 2024
@MosheEichler MosheEichler merged commit cad55f3 into master Oct 1, 2024
16 checks passed
@MosheEichler MosheEichler deleted the BugFix/XSUP-41283/CyberArkIdentity-improve-the-error-message branch October 1, 2024 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants