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

Update Listen Port description #31907

Merged
merged 29 commits into from Mar 17, 2024
Merged

Update Listen Port description #31907

merged 29 commits into from Mar 17, 2024

Conversation

ShirleyDenkberg
Copy link
Contributor

@ShirleyDenkberg ShirleyDenkberg commented Jan 2, 2024

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

https://jira-dc.paloaltonetworks.com/browse/CRTXDOC-1811

Description

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

Must have

  • Tests
  • Documentation

Copy link

github-actions bot commented Jan 4, 2024

Your contributed PublishList pack has been modified on files:

Packs/PublishList/Integrations/PublishList/PublishList_description.md
Please review the changes here

Copy link

github-actions bot commented Jan 4, 2024

Your contributed SimpleAPIProxy pack has been modified on files:

Packs/SimpleAPIProxy/Integrations/SimpleAPIProxy/SimpleAPIProxy_description.md
Please review the changes here

Copy link
Contributor

@ShahafBenYakir ShahafBenYakir left a comment

Choose a reason for hiding this comment

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

Ping me when the PR is ready and I will fix the pre-commit errors

Packs/EDL/Integrations/EDL/EDL_description.md Outdated Show resolved Hide resolved
Packs/EDL/Integrations/EDL/README.md Outdated Show resolved Hide resolved

- (For Cortex XSOAR 6.x) `https://*<xsoar_address>*/instance/execute/*<instance_name>*`
- (For Cortex XSOAR 8 or Cortex XSIAM) `https://ext-<tenant>.crtx.<region>.paloaltonetworks.com/xsoar/instance/execute/<instance-name>`
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not supported?

Copy link
Contributor

Choose a reason for hiding this comment

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

Not asking on the web access, I am asking about this part that was deleted:

   - (For Cortex XSOAR 6.x) `https://*<xsoar_address>*/instance/execute/*<instance_name>*`
   - (For Cortex XSOAR 8 or Cortex XSIAM) `https://ext-<tenant>.crtx.<region>.paloaltonetworks.com/xsoar/instance/execute/<instance-name>`

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ShahafBenYakir See the note I added at the beginning of this section.

@ShahafBenYakir
Copy link
Contributor

@ShirleyDenkberg Should be ok to merge once you are ready

@@ -35,10 +35,10 @@ curl -v -u test:password https://edl-my-xsiam-subdomain.us.paloaltonetworks.com/
**Note**: When using an instance that is running on an engine, you can access only with the `Listen Port`. Go to **http://<cortex-xsoar-server-address>:<listen_port>**.

1. To access the **Generic Export Indicators Service** by instance name, set up the **username** and **password** values in the **External Dynamic List Integration** page (**Settings & info** > **SETTINGS** > **Long Running Integrations**).
2. You can access the External Dynamic List at the following url: `https://edl-<cortex-xsoar-address>/xsoar/instance/execute/<instance-name>`.
2. You can access the External Dynamic List at the following URL: `https://ext-<cortex-xsoar-address>/xsoar/instance/execute/<instance-name>`. Note: You cannot access the External Dynamic List via a web browser.
Copy link
Contributor

Choose a reason for hiding this comment

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

Why can't he access it via a browser? its a text file. As far as I know both ext and edl will work

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not for 8 or xsiam

@ShirleyDenkberg ShirleyDenkberg merged commit c7b6caf into master Mar 17, 2024
16 of 17 checks passed
@ShirleyDenkberg ShirleyDenkberg deleted the ShirleyDenkberg-patch-1 branch March 17, 2024 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants