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

Feature: Infoblox manage session with cookies in integration storage #37240

Open
wants to merge 10 commits into
base: contrib/jesusgpo_contrib/infoblox-manage-session-cookies-in-context
Choose a base branch
from

Conversation

jesusgpo
Copy link
Contributor

Contributing to Cortex XSOAR Content

Status

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

Related Issues

N/A

Description

Now the infoblock integration is doing the authentication for each command execution, this has a very high impact on infoblox, that's why the cookies are stored in the context of the integration and loaded before each call, avoiding to login unless the cookies expire.

@jesusgpo
Copy link
Contributor Author

Hi may, yesterday I attached you the demo video by slack

@jesusgpo
Copy link
Contributor Author

Hi @MLainer1

I have just updated the release, thank you for your help.

Kind regards

@Shellyber
Copy link
Contributor

For the Reviewer: Trigger build request has been accepted for this contribution PR.

@Shellyber
Copy link
Contributor

For the Reviewer: Successfully created a pipeline in GitLab with url: https://gitlab.xdr.pan.local/xdr/cortex-content/content/-/pipelines/1717709

@Shellyber Shellyber removed the ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. label Nov 28, 2024
@ShirleyDenkberg ShirleyDenkberg self-assigned this Dec 3, 2024
@ShirleyDenkberg
Copy link
Contributor

@jesusgpo Doc review completed.

@CLAassistant
Copy link

CLAassistant commented Dec 3, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 3 committers have signed the CLA.

❌ MLainer1
❌ jesusgpo
❌ samuelFain


MLainer1 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 3 committers have signed the CLA.

❌ MLainer1
❌ jesusgpo
❌ samuelFain


MLainer1 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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 External PR Partner pending-contributor The PR is pending the response of its creator post-demo Xsoar Support Level Indicates that the contribution is for XSOAR supported pack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants