XSUP 75257 - Add sanitize_large_ints function to handle large ThreatConnect object IDs

[lironcohen272]

Related Issues

fixes: https://jira-dc.paloaltonetworks.com/browse/XSUP-75257

Description

Fixed an issue where large ThreatConnect object IDs could lose numerical precision when processed by XSOAR, resulting in incorrect IDs being used by downstream commands. Large integer values are now returned as strings to preserve their accuracy.

[content-bot]

🤖 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

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
Packs/ThreatConnect/Integrations/ThreatConnectV3
ThreatConnectV3.py	535	89	83%	147, 153, 163, 171, 174–176, 181–182, 188–189, 191, 196, 198–201, 204, 211–214, 230, 234–236, 243–244, 272, 274, 326, 416–423, 497, 502–504, 513–514, 521, 558–559, 753, 819, 824–825, 827–828, 880–881, 895–896, 911, 913, 923, 1956, 2043–2047, 2197–2201, 2312–2316, 2338, 2363–2373
TOTAL	535	89	83%

Tests	Skipped	Failures	Errors	Time
47	0 💤	0 ❌	0 🔥	3.331s ⏱️

[marketplace-ai-reviewer]

🤖 Analysis started. Please wait for results...

[marketplace-ai-reviewer]

🤖 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]

Hi! Thanks for your work on the ThreatConnect integration updates. I've left a few suggestions to help polish the code, primarily updating the Client class to inherit from ContentClient instead of BaseClient. There are also a few minor notes regarding adding type hints, handling negative large integers, expanding test coverage, and adjusting the release notes for Cortex branding. Great job overall!

Additionally, please address the following file-level notes:

• Packs/ThreatConnect/Integrations/ThreatConnectV3/ThreatConnectV3.py: The Client class inherits from BaseClient instead of ContentClient.

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

[content-bot]

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: 4d529b0406d1d60d

📋 Triage Report
💡 Resolutions are available in the full report.

⚠️ AI-generated triage. Validate before acting.

[content-bot]

Validate summary
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? ✅