Skip to content

CIAC-17723 | fix(OpenAI): send Bearer scheme on Compliance API auth header - #45633

Merged
DanielTal87 merged 5 commits into
masterfrom
CIAC-17723
Aug 25, 2026
Merged

CIAC-17723 | fix(OpenAI): send Bearer scheme on Compliance API auth header#45633
DanielTal87 merged 5 commits into
masterfrom
CIAC-17723

Conversation

@DanielTal87

@DanielTal87 DanielTal87 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Status

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

Related Issues

fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-17723

Must have

  • Tests
  • Documentation

@DanielTal87
DanielTal87 requested a review from JasBeilin as a code owner August 24, 2026 13:46
@content-bot

content-bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

🤖 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

@DanielTal87 DanielTal87 self-assigned this Aug 24, 2026
@DanielTal87 DanielTal87 added docs-approved 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. labels Aug 24, 2026
@DanielTal87

DanielTal87 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor Author

@marketplace-ai-reviewer start review

@github-actions

github-actions Bot commented Aug 24, 2026 •
edited
Loading

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/OpenAI/Integrations/OpenAiChatGPTV3
   OpenAiChatGPTV3.py9424595%560, 608–610, 754, 758, 762, 830, 838, 848, 910–915, 967, 1018, 1022, 1050, 1125, 1177, 1182–1184, 1187, 1190, 1422, 1452–1453, 1459, 1611, 2060–2063, 2068–2070, 2144–2145, 2158–2159, 2318, 2396
TOTAL9424595% 

Tests Skipped Failures Errors Time
224 0 💤 0 ❌ 0 🔥 8.595s ⏱️

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

marketplace-ai-reviewer commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

🤖 Analysis started. Please wait for results...

marketplace-ai-reviewer

This comment was marked as outdated.

Sign in to view

@DanielTal87 DanielTal87 changed the title fix(OpenAI): send Bearer scheme on Compliance API auth header (CIAC-1… fix(OpenAI): send Bearer scheme on Compliance API auth header Aug 24, 2026
dtroushinsky
dtroushinsky approved these changes Aug 24, 2026
View reviewed changes
Comment thread Packs/OpenAI/Integrations/OpenAiChatGPTV3/OpenAiChatGPTV3.py Outdated
Comment thread Packs/OpenAI/Integrations/OpenAiChatGPTV3/OpenAiChatGPTV3_test.py Outdated
@DanielTal87 DanielTal87 changed the title fix(OpenAI): send Bearer scheme on Compliance API auth header CIAC-17723 | fix(OpenAI): send Bearer scheme on Compliance API auth header Aug 24, 2026
@DanielTal87

DanielTal87 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor Author

@marketplace-ai-reviewer start review

@marketplace-ai-reviewer

marketplace-ai-reviewer commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

🤖 Analysis started. Please wait for results...

@marketplace-ai-reviewer

marketplace-ai-reviewer commented Aug 24, 2026

Copy link
Copy Markdown
Contributor
🤖 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
marketplace-ai-reviewer reviewed Aug 24, 2026
View reviewed changes

@marketplace-ai-reviewer marketplace-ai-reviewer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution! I've taken a look at the PR and found a couple of minor things to adjust. Specifically, please ensure the Event Collector's first_fetch defaults to 1 minute ago, and expand the release notes to clearly cover all the changes made in this PR. Let me know if you have any questions!

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

Comment thread Packs/OpenAI/Integrations/OpenAiChatGPTV3/OpenAiChatGPTV3.py Outdated
Comment thread Packs/OpenAI/ReleaseNotes/2_1_5.md Outdated
@content-bot

This comment has been minimized.

Sign in to view
@DanielTal87
DanielTal87 enabled auto-merge (squash) August 24, 2026 16:01
@content-bot

content-bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

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

@content-bot

content-bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: success
Report ID: 76eb4ddec6b55279

📋 Triage Report

⚠️ AI-generated triage. Validate before acting.

@DanielTal87
DanielTal87 merged commit 6b916f1 into master Aug 25, 2026
26 checks passed
@DanielTal87
DanielTal87 deleted the CIAC-17723 branch August 25, 2026 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marketplace-ai-reviewer marketplace-ai-reviewer marketplace-ai-reviewer left review comments
@dtroushinsky dtroushinsky dtroushinsky approved these changes
@JasBeilin JasBeilin Awaiting requested review from JasBeilin JasBeilin is a code owner

Assignees

@DanielTal87 DanielTal87

Labels

docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@DanielTal87 @content-bot @marketplace-ai-reviewer @dtroushinsky