Msg bug xsup71559

[nbensalm-palo]

Status

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

Related Issues

https://jira-dc.paloaltonetworks.com/browse/XSUP-71559

Description

Fixed an issue where downloading an eDiscovery export file caused subsequent commands to fail with a Unauthorized error.

Must have

• Tests
• Documentation

[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

[marketplace-ai-reviewer]

🤖 Analysis started. Please wait for results...

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity
MicrosoftGraphSecurity.py	945	254	73%	69, 72–73, 76–78, 81–83, 86–87, 90–92, 95–97, 100–102, 105–108, 111–112, 126–127, 130–131, 134–135, 138–139, 142–143, 146–147, 150–153, 156–157, 162–163, 176, 180–181, 186, 189–192, 195–197, 200–201, 207, 221–224, 227–228, 231–233, 236–238, 240–242, 244, 254–256, 259, 261, 264–265, 286–287, 292, 309–310, 336–337, 341, 364–367, 384–385, 402–405, 422–423, 456–457, 467–468, 479, 486, 489, 492, 497, 502, 509, 512, 519, 522, 528, 531–533, 538–543, 545, 562, 584–585, 624, 634, 637, 680–683, 707, 709, 711, 713, 765–768, 770–771, 784, 812, 1023, 1055, 1093–1100, 1136–1140, 1271–1283, 1298–1304, 1332–1333, 1374, 1377, 1381–1382, 1402–1403, 1407–1408, 1416, 1425, 1434, 1444, 1453–1456, 1460, 1464, 1481, 1483, 1593, 1602–1603, 1606–1607, 1618, 1693, 1767, 1814, 1899, 1907, 1915, 1939–1940, 2150, 2220–2222, 2224, 2261, 2294, 2327, 2358, 2371, 2379, 2422–2423, 2427–2428, 2501–2505, 2512, 2514, 2516, 2518, 2520, 2522, 2524, 2526, 2528, 2532, 2535, 2540–2541
TOTAL	945	254	73%

Tests	Skipped	Failures	Errors	Time
97	0 💤	0 ❌	0 🔥	3.487s ⏱️

[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 there! Thanks for your contribution to the Microsoft Graph Security pack. I've reviewed the code and left a few minor suggestions to help polish things up. The feedback mostly focuses on extracting hardcoded values, ensuring test mock accuracy, and a couple of quick metadata and typo tweaks. Great work overall!

Additionally, please address the following file-level notes:

• Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.py: Consider extracting the hardcoded Purview scope "b26e684c-5068-4120-a679-64a5d2c909d9/.default" used in download_export_file into a constant at the top of the file (e.g., PURVIEW_EDISCOVERY_SCOPE).
• Packs/MicrosoftGraphSecurity/pack_metadata.json: The vendor name (e.g., "Microsoft") is missing from the keywords list. Please add it to improve searchability.

@Shellyber, @noydavidi, @nbensalm-palo please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

[noydavidi]

Great job getting to the bottom of that bug! 🔥

[content-bot]

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: 9a5e2ca2aa796251

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

⚠️ AI-generated triage. Validate before acting.

[content-bot]

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