Fix/XSUP-69905/CP-HEC Updated the authentication method

[MosheEichler]

Status

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

Related Issues

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

Description

Moved clientId from the header into the JSON body alongside accessKey

Swagger to the UserControl request
https://app.swaggerhub.com/apis-docs/Check-Point/infinity-portal-api/1.0.6#/User%20Control/post_auth_external

Screenshot

Must have

• Tests
• Documentation

[github-actions]

Your contributed CheckPointHEC pack has been modified on files:

Packs/CheckPointHEC/ReleaseNotes/1_1_16.md
Packs/CheckPointHEC/Integrations/CheckPointHEC/CheckPointHEC_test.py
Packs/CheckPointHEC/Integrations/CheckPointHEC/CheckPointHEC.py
Packs/CheckPointHEC/pack_metadata.json
Please review the changes here

• @chkp-arvidb
• @chkp-christiang

[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/CheckPointHEC/Integrations/CheckPointHEC
CheckPointHEC.py	802	99	87%	215, 253, 261, 265, 267, 271, 273, 275, 277, 279, 281, 285, 289, 293, 295, 299, 301, 303, 305, 309, 311, 315, 322, 324, 497, 506, 598, 601, 604, 622–623, 625, 642, 661, 666, 681–682, 684, 694–695, 697, 699, 701–702, 704–706, 708–709, 719–720, 723, 799, 802, 821, 826, 899, 912, 922, 968, 974, 1365–1366, 1368–1371, 1378, 1437–1443, 1445, 1448–1450, 1457, 1474–1475, 1477–1480, 1487, 1546–1552, 1554, 1557–1559, 1566
TOTAL	802	99	87%

Tests	Skipped	Failures	Errors	Time
88	0 💤	0 ❌	0 🔥	6.579s ⏱️

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

[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 there, thanks for your contribution! I've reviewed your PR and just have a minor formatting note. Please ensure the API name and error message in the release notes are bolded to align with our entity styling guidelines. Great work overall!

@marketplace-ai-reviewer, @DeanArbel, @MosheEichler please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

[marketplace-ai-reviewer]

Please apply the correct entity styling for the API name and the error message.

According to the Entity Styling guidelines, proper nouns and literal values surfaced to the user (such as error messages) should be formatted in bold.

Suggested change

	- Fixed an issue where the Check Point Infinity API authentication failed with "Client id not found".
	- Fixed an issue where the **Check Point Infinity API** authentication failed with **"Client id not found"**.

[content-bot]

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: success
Report ID: 9d53547e4d0980a8

📋 Triage Report

⚠️ AI-generated triage. Validate before acting.