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

fix(SEKOIAIntelligenceCenter): fix endpoint for test command #24437

Conversation

PierrickV
Copy link
Contributor

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

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

Related Issues

Wrong endpoint was called for test-module resulting in a Token revoked error

Description

Fix issue

Screenshots

Before (red) after (green)
Screenshot from 2023-02-07 16-10-09

Minimum version of Cortex XSOAR

  • 6.0.0
  • 6.1.0
  • 6.2.0
  • 6.5.0

Does it break backward compatibility?

  • Yes
    • Further details:
  • No

Must have

  • Tests
  • Documentation

@content-bot content-bot added the Contribution Thank you! Contributions are always welcome! label Feb 7, 2023
@content-bot content-bot changed the base branch from master to contrib/SEKOIA-IO_sekoia-io-cti-fix-wrong-test-endpoint February 7, 2023 15:22
@content-bot
Copy link
Collaborator

Thank you for your contribution. Your generosity and caring are unrivaled! Make sure to register your contribution by filling the Contribution Registration form, so our content wizard @yaakovpraisler will know he can start review the proposed changes.

@content-bot content-bot added Contribution Form Filled Whether contribution form filled or not. Partner labels Feb 7, 2023
…' into sekoia-io-cti-fix-wrong-test-endpoint
Copy link
Contributor

@yaakovpraisler yaakovpraisler left a comment

Choose a reason for hiding this comment

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

Hi @PierrickV - Thank for fixing it! good job :)
Please bump the version of the pack in the pack_metadata.json file and we'll continue to merge.

Packs/SEKOIAIntelligenceCenter/ReleaseNotes/1_2_1.md Outdated Show resolved Hide resolved
@yaakovpraisler yaakovpraisler added the pending-contributor The PR is pending the response of its creator label Feb 8, 2023
@PierrickV PierrickV force-pushed the sekoia-io-cti-fix-wrong-test-endpoint branch from 2908083 to a97444d Compare February 8, 2023 13:12
Co-authored-by: Yaakov Praisler <59408745+yaakovpraisler@users.noreply.github.com>
@PierrickV PierrickV force-pushed the sekoia-io-cti-fix-wrong-test-endpoint branch from a97444d to 6739944 Compare February 8, 2023 13:13
@PierrickV
Copy link
Contributor Author

@yaakovpraisler : thank you for taking the time to review this work! I applied your suggestions 😸

…' into sekoia-io-cti-fix-wrong-test-endpoint
@yaakovpraisler yaakovpraisler merged commit 81fdd3a into demisto:contrib/SEKOIA-IO_sekoia-io-cti-fix-wrong-test-endpoint Feb 8, 2023
yaakovpraisler added a commit that referenced this pull request Feb 8, 2023
…#24485)

* fix(SEKOIAIntelligenceCenter): fix endpoint for test command

* Apply suggestions from code review by @yaakovpraisler



---------

Co-authored-by: lilyus <8960084+PierrickV@users.noreply.github.com>
Co-authored-by: Yaakov Praisler <59408745+yaakovpraisler@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contribution Form Filled Whether contribution form filled or not. Contribution Thank you! Contributions are always welcome! docs-approved Partner pending-contributor The PR is pending the response of its creator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants