Core api bug by merit-maita · Pull Request #43725 · demisto/content

merit-maita · 2026-03-29T13:01:20Z

Status

In Progress
Ready
In Hold - (Reason for hold)

Related Issues

Fixes: https://jira-dc.paloaltonetworks.com/browse/XSUP-64300

Description

Fixed an issue where the "sendRequest" method was missing the retry mechanism.

Must have

Tests
Documentation

content-bot · 2026-03-29T13:02:37Z

🤖 AI-Powered Code Review Available

You can leverage AI-powered code review to assist with this PR!

Available Commands:

@marketplace-ai-reviewer start review - Initiate a full AI code review
@marketplace-ai-reviewer re-review - Incremental review for new commits

marketplace-ai-reviewer · 2026-03-29T13:11:10Z

🤖 Analysis started. Please wait for results...

marketplace-ai-reviewer · 2026-03-29T13:22:56Z

🤖 AI Review Disclaimer

This review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause.

marketplace-ai-reviewer

Thanks for your contribution! I've reviewed the changes and have a few suggestions to improve the integration's safety and reliability. Mainly, please ensure we aren't logging full request or response objects to prevent leaking sensitive data, add unit tests for the new retry logic, and update the release notes to accurately reflect these updates. Let me know if you need any help with these!

Additionally, please address the following file-level notes:

Packs/DemistoRESTAPI/Integrations/CoreRESTAPI/CoreRESTAPI.js: Please add unit tests for the new retry logic in sendRequest and sendMultipart.

please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

Co-authored-by: merit-maita <49760643+merit-maita@users.noreply.github.com>

content-bot · 2026-04-13T07:37:02Z

Validate summary
The following errors were thrown as a part of this pr: .
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

Verdict: PR can be force merged from validate perspective? ✅

merit-maita · 2026-04-13T11:14:51Z

The PR needs to be force merged as two test playbooks (Detect & Manage Phishing Campaigns - Test and Grafana-Test ) are failing with known issues.
@kamalq97 @yuvalbenshalom FYI

* added the retry mechanism * added rn * fix * Apply suggestions from code review Co-authored-by: merit-maita <49760643+merit-maita@users.noreply.github.com> --------- Co-authored-by: Kamal Qarain <45042524+kamalq97@users.noreply.github.com>

@kobymeir

…43855) * fixes * RN * Ruff fixes * pre-commit fixes * CRTX-240216 - Enhancement - msft_azure_ad_raw (#43786) * update xif * update xif with sign-in log enhancements and release notes * fix: add missing location field to msft_azure_ad_raw schema * Core api bug (#43725) * added the retry mechanism * added rn * fix * Apply suggestions from code review Co-authored-by: merit-maita <49760643+merit-maita@users.noreply.github.com> --------- Co-authored-by: Kamal Qarain <45042524+kamalq97@users.noreply.github.com> * TelemtryCortexRunScriptOnEndpoint (#43472) * TelemtryCortexRunScriptOnEndpoint * rn * ai reviwer * RN * Fixed for F5 ASM cs3 field and improving ipv4/6 implementation (#43845) * fixed cs3 fields * fixed ipv4 and ipv6 fields * added release notes * removed alter * CIAC-16216 - marketplacev2 content items should show in platform (#43808) * CIAC-16216 * CIAC-16216 * Update 1_0_1.md * Update docker image * Update readme files * Add docker update to release notes * Add docker update to release notes * XSUP-66843_add_azure_resourceGraph_gov_support (#43805) * XSUP-66843_add_azure_resourceGraph_gov_support * ai review update * doc review and fixes * empty commit * cr fixes * fix adf comment handling (#43813) * fix adf comment handling * RN * unittests * fix ai review * fix * docker image * fix RN * Bump pack from version Jira to 3.3.28. --------- Co-authored-by: Content Bot <bot@demisto.com> * Fix Crtx 232255 (#43713) * Fix caps in owner file * licensing fix * RN * Address CR * Bump pack from version CortexXDR to 6.3.14. * Fix secrets detection issue in CortexXDRIR_test.py * Fix validations * Address CR * Address CR * Docker image * Fix validations * fix validation * rn * Bump pack from version CortexXDR to 6.3.15. * rn * Apply suggestion from @kobymeir Co-authored-by: Koby Meir <kobymeir@users.noreply.github.com> * Update Packs/CortexXDR/ReleaseNotes/6_3_15.md Co-authored-by: Arad Carmi <62752352+AradCarmi@users.noreply.github.com> * rn * RN --------- Co-authored-by: Content Bot <bot@demisto.com> Co-authored-by: Koby Meir <kobymeir@users.noreply.github.com> Co-authored-by: Arad Carmi <62752352+AradCarmi@users.noreply.github.com> * EWSO365: Fix duplicate detection for malformed Message-ID bracket variants (#43811) * unitets + dedup function modify * fix is_item_duplicate comments * add release notes * add release notes * fix is_item_duplicate comments * fix after ai review * add debug messages * UnEscapeURLs: Fixed truncating URLs issue (#43830) * the fix * RN * fix validation * Update UnEscapeURLs.yml * Add Superna Zero Trust integration pack (#43826) * Add Superna Zero Trust integration pack (#43685) * Add Superna Zero Trust integration pack New partner pack for ransomware response automation: - Critical path snapshot creation - User NAS lockout/unlock capabilities - HTTP 429 error handling for rate limiting - 4 automated playbooks with integration availability checks - Complete documentation and playbook screenshots Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Address PR review feedback for Superna Zero Trust pack Unit Tests: - Renamed snapshot_cmd -> snapshot_critical_paths_command - Renamed lockout_cmd -> lockout_user_command - Renamed unlock_cmd -> unlock_user_command - Updated main() dispatch to use new function names - Fixed SupernaZeroTrust_test.py: Client init now uses api_key= (not headers=) - Fixed test assertions to match actual outputs structure (outputs_prefix, outputs keys) Integration YAML (SupernaZeroTrust.yml): - Added sectionorder: [Connect] to satisfy ST111 - Updated dockerimage from demisto/python3:3.10.13.86272 to demisto/python3:3.12.13.7444307 (DO106) New File: - Created SupernaZeroTrust_description.md with prerequisites, API key instructions, and parameter table (DS104) Documentation: - Rewrote Integrations/SupernaZeroTrust/README.md with full command documentation including arguments, context outputs, and examples for all 3 commands (RM110) - Rewrote Packs/SupernaZeroTrust/README.md with full pack overview, use cases, contents table, configuration, and support info (RM104) Pack Metadata: - Fixed categories: replaced "Incident Response" (invalid) with "Endpoint" (PA103) Validation: - Added ignore=BA129 to .pack-ignore for superna-zt-lockout-user and superna-zt-unlock-user compliance policy warnings, as recommended by maintainers (BA129) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Fix ST111 validation, unit test trailing slash, and suppress RM104/RM110 - ST111: Added section: Connect to all 4 parameters in SupernaZeroTrust.yml (base_url, credentials, insecure, proxy) - Unit test: Fixed test_client_initialization trailing slash assertion to use rstrip('/') for compatibility across demisto-sdk versions - RM104/RM110: Added ignores to .pack-ignore (suppressible warnings) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: Kamal Qarain <45042524+kamalq97@users.noreply.github.com> * Fix pre-commit issues --------- Co-authored-by: Andrew MacKay <andrew.mackay@superna.net> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: Kamal Qarain <45042524+kamalq97@users.noreply.github.com> Co-authored-by: Kamal Qarain <kqarain@paloaltonetworks.com> * fix validate * Add parameter * ruff fixes --------- Co-authored-by: akshotiamit-pa <aakshoti@paloaltonetworks.com> Co-authored-by: merit-maita <49760643+merit-maita@users.noreply.github.com> Co-authored-by: Kamal Qarain <45042524+kamalq97@users.noreply.github.com> Co-authored-by: hyaffe839 <hyaffe@paloaltonetworks.com> Co-authored-by: sharonfi99 <147984773+sharonfi99@users.noreply.github.com> Co-authored-by: dtroushinsky <dtroushinsky@paloaltonetworks.com> Co-authored-by: masulin97 <masulin@paloaltonetworks.com> Co-authored-by: inbalapt1 <164751454+inbalapt1@users.noreply.github.com> Co-authored-by: Content Bot <bot@demisto.com> Co-authored-by: Tal Zichlinsky <35036457+talzich@users.noreply.github.com> Co-authored-by: Koby Meir <kobymeir@users.noreply.github.com> Co-authored-by: Arad Carmi <62752352+AradCarmi@users.noreply.github.com> Co-authored-by: almog2296 <alabudi@paloaltonetworks.com> Co-authored-by: Israel Lappe <79846863+ilappe@users.noreply.github.com> Co-authored-by: Content Bot <55035720+content-bot@users.noreply.github.com> Co-authored-by: Andrew MacKay <andrew.mackay@superna.net> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: Kamal Qarain <kqarain@paloaltonetworks.com>

* added the retry mechanism * added rn * fix * Apply suggestions from code review Co-authored-by: merit-maita <49760643+merit-maita@users.noreply.github.com> --------- Co-authored-by: Kamal Qarain <45042524+kamalq97@users.noreply.github.com>

@kobymeir

…emisto#43855) * fixes * RN * Ruff fixes * pre-commit fixes * CRTX-240216 - Enhancement - msft_azure_ad_raw (demisto#43786) * update xif * update xif with sign-in log enhancements and release notes * fix: add missing location field to msft_azure_ad_raw schema * Core api bug (demisto#43725) * added the retry mechanism * added rn * fix * Apply suggestions from code review Co-authored-by: merit-maita <49760643+merit-maita@users.noreply.github.com> --------- Co-authored-by: Kamal Qarain <45042524+kamalq97@users.noreply.github.com> * TelemtryCortexRunScriptOnEndpoint (demisto#43472) * TelemtryCortexRunScriptOnEndpoint * rn * ai reviwer * RN * Fixed for F5 ASM cs3 field and improving ipv4/6 implementation (demisto#43845) * fixed cs3 fields * fixed ipv4 and ipv6 fields * added release notes * removed alter * CIAC-16216 - marketplacev2 content items should show in platform (demisto#43808) * CIAC-16216 * CIAC-16216 * Update 1_0_1.md * Update docker image * Update readme files * Add docker update to release notes * Add docker update to release notes * XSUP-66843_add_azure_resourceGraph_gov_support (demisto#43805) * XSUP-66843_add_azure_resourceGraph_gov_support * ai review update * doc review and fixes * empty commit * cr fixes * fix adf comment handling (demisto#43813) * fix adf comment handling * RN * unittests * fix ai review * fix * docker image * fix RN * Bump pack from version Jira to 3.3.28. --------- Co-authored-by: Content Bot <bot@demisto.com> * Fix Crtx 232255 (demisto#43713) * Fix caps in owner file * licensing fix * RN * Address CR * Bump pack from version CortexXDR to 6.3.14. * Fix secrets detection issue in CortexXDRIR_test.py * Fix validations * Address CR * Address CR * Docker image * Fix validations * fix validation * rn * Bump pack from version CortexXDR to 6.3.15. * rn * Apply suggestion from @kobymeir Co-authored-by: Koby Meir <kobymeir@users.noreply.github.com> * Update Packs/CortexXDR/ReleaseNotes/6_3_15.md Co-authored-by: Arad Carmi <62752352+AradCarmi@users.noreply.github.com> * rn * RN --------- Co-authored-by: Content Bot <bot@demisto.com> Co-authored-by: Koby Meir <kobymeir@users.noreply.github.com> Co-authored-by: Arad Carmi <62752352+AradCarmi@users.noreply.github.com> * EWSO365: Fix duplicate detection for malformed Message-ID bracket variants (demisto#43811) * unitets + dedup function modify * fix is_item_duplicate comments * add release notes * add release notes * fix is_item_duplicate comments * fix after ai review * add debug messages * UnEscapeURLs: Fixed truncating URLs issue (demisto#43830) * the fix * RN * fix validation * Update UnEscapeURLs.yml * Add Superna Zero Trust integration pack (demisto#43826) * Add Superna Zero Trust integration pack (demisto#43685) * Add Superna Zero Trust integration pack New partner pack for ransomware response automation: - Critical path snapshot creation - User NAS lockout/unlock capabilities - HTTP 429 error handling for rate limiting - 4 automated playbooks with integration availability checks - Complete documentation and playbook screenshots Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Address PR review feedback for Superna Zero Trust pack Unit Tests: - Renamed snapshot_cmd -> snapshot_critical_paths_command - Renamed lockout_cmd -> lockout_user_command - Renamed unlock_cmd -> unlock_user_command - Updated main() dispatch to use new function names - Fixed SupernaZeroTrust_test.py: Client init now uses api_key= (not headers=) - Fixed test assertions to match actual outputs structure (outputs_prefix, outputs keys) Integration YAML (SupernaZeroTrust.yml): - Added sectionorder: [Connect] to satisfy ST111 - Updated dockerimage from demisto/python3:3.10.13.86272 to demisto/python3:3.12.13.7444307 (DO106) New File: - Created SupernaZeroTrust_description.md with prerequisites, API key instructions, and parameter table (DS104) Documentation: - Rewrote Integrations/SupernaZeroTrust/README.md with full command documentation including arguments, context outputs, and examples for all 3 commands (RM110) - Rewrote Packs/SupernaZeroTrust/README.md with full pack overview, use cases, contents table, configuration, and support info (RM104) Pack Metadata: - Fixed categories: replaced "Incident Response" (invalid) with "Endpoint" (PA103) Validation: - Added ignore=BA129 to .pack-ignore for superna-zt-lockout-user and superna-zt-unlock-user compliance policy warnings, as recommended by maintainers (BA129) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Fix ST111 validation, unit test trailing slash, and suppress RM104/RM110 - ST111: Added section: Connect to all 4 parameters in SupernaZeroTrust.yml (base_url, credentials, insecure, proxy) - Unit test: Fixed test_client_initialization trailing slash assertion to use rstrip('/') for compatibility across demisto-sdk versions - RM104/RM110: Added ignores to .pack-ignore (suppressible warnings) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: Kamal Qarain <45042524+kamalq97@users.noreply.github.com> * Fix pre-commit issues --------- Co-authored-by: Andrew MacKay <andrew.mackay@superna.net> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: Kamal Qarain <45042524+kamalq97@users.noreply.github.com> Co-authored-by: Kamal Qarain <kqarain@paloaltonetworks.com> * fix validate * Add parameter * ruff fixes --------- Co-authored-by: akshotiamit-pa <aakshoti@paloaltonetworks.com> Co-authored-by: merit-maita <49760643+merit-maita@users.noreply.github.com> Co-authored-by: Kamal Qarain <45042524+kamalq97@users.noreply.github.com> Co-authored-by: hyaffe839 <hyaffe@paloaltonetworks.com> Co-authored-by: sharonfi99 <147984773+sharonfi99@users.noreply.github.com> Co-authored-by: dtroushinsky <dtroushinsky@paloaltonetworks.com> Co-authored-by: masulin97 <masulin@paloaltonetworks.com> Co-authored-by: inbalapt1 <164751454+inbalapt1@users.noreply.github.com> Co-authored-by: Content Bot <bot@demisto.com> Co-authored-by: Tal Zichlinsky <35036457+talzich@users.noreply.github.com> Co-authored-by: Koby Meir <kobymeir@users.noreply.github.com> Co-authored-by: Arad Carmi <62752352+AradCarmi@users.noreply.github.com> Co-authored-by: almog2296 <alabudi@paloaltonetworks.com> Co-authored-by: Israel Lappe <79846863+ilappe@users.noreply.github.com> Co-authored-by: Content Bot <55035720+content-bot@users.noreply.github.com> Co-authored-by: Andrew MacKay <andrew.mackay@superna.net> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: Kamal Qarain <kqarain@paloaltonetworks.com>

merit-maita added 2 commits March 29, 2026 15:42

added the retry mechanism

ab90376

added rn

17270c9

merit-maita added ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. docs-approved labels Mar 29, 2026

marketplace-ai-reviewer removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Mar 29, 2026

marketplace-ai-reviewer reviewed Mar 29, 2026

View reviewed changes

Comment thread Packs/DemistoRESTAPI/Integrations/CoreRESTAPI/CoreRESTAPI.js

Comment thread Packs/DemistoRESTAPI/Integrations/CoreRESTAPI/CoreRESTAPI.js

Comment thread Packs/DemistoRESTAPI/ReleaseNotes/1_4_1.md Outdated

fix

9a7dd0e

merit-maita requested a review from kamalq97 March 30, 2026 08:57

Merge branch 'master' into core-api-bug

a76b3d3

kamalq97 reviewed Apr 12, 2026

View reviewed changes

Comment thread Packs/DemistoRESTAPI/ReleaseNotes/1_4_1.md Outdated

merit-maita commented Apr 12, 2026

View reviewed changes

Comment thread Packs/DemistoRESTAPI/ReleaseNotes/1_4_1.md Outdated

merit-maita added 2 commits April 12, 2026 12:05

Apply suggestions from code review

1f817ff

Co-authored-by: merit-maita <49760643+merit-maita@users.noreply.github.com>

Merge branch 'master' into core-api-bug

4ffb555

merit-maita requested a review from kamalq97 April 12, 2026 09:10

kamalq97 approved these changes Apr 12, 2026

View reviewed changes

merit-maita added 2 commits April 12, 2026 16:16

Merge branch 'master' into core-api-bug

7b19245

Merge branch 'master' into core-api-bug

48bc0e5

merit-maita added the ForceMerge Forcing the merge of the PR despite the build status label Apr 13, 2026

merit-maita requested a review from yuvalbenshalom April 13, 2026 11:24

yuvalbenshalom approved these changes Apr 13, 2026

View reviewed changes

yuvalbenshalom merged commit 858e337 into master Apr 14, 2026
29 of 30 checks passed

yuvalbenshalom deleted the core-api-bug branch April 14, 2026 06:38

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Core api bug#43725

Core api bug#43725
yuvalbenshalom merged 8 commits intomasterfrom
core-api-bug

merit-maita commented Mar 29, 2026 •

edited

Loading

Uh oh!

content-bot commented Mar 29, 2026

Uh oh!

marketplace-ai-reviewer commented Mar 29, 2026

Uh oh!

marketplace-ai-reviewer commented Mar 29, 2026

Uh oh!

marketplace-ai-reviewer left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

content-bot commented Apr 13, 2026

Uh oh!

merit-maita commented Apr 13, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

merit-maita commented Mar 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Status

Related Issues

Description

Must have

Uh oh!

content-bot commented Mar 29, 2026

🤖 AI-Powered Code Review Available

Uh oh!

marketplace-ai-reviewer commented Mar 29, 2026

Uh oh!

marketplace-ai-reviewer commented Mar 29, 2026

Uh oh!

marketplace-ai-reviewer left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

content-bot commented Apr 13, 2026

Uh oh!

merit-maita commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

merit-maita commented Mar 29, 2026 •

edited

Loading

merit-maita commented Apr 13, 2026 •

edited

Loading