Skip to content

Add API Endpoints to PwnedV2#43829

Draft
TheL0L wants to merge 15 commits into
masterfrom
syakima/CIAC-15948/pwnd-add-api-endpoints
Draft

Add API Endpoints to PwnedV2#43829
TheL0L wants to merge 15 commits into
masterfrom
syakima/CIAC-15948/pwnd-add-api-endpoints

Conversation

@TheL0L
Copy link
Copy Markdown
Contributor

@TheL0L TheL0L commented Apr 13, 2026

Related Issues

fixes: CIAC-15948

Description

Add new commands to support additional API endpoints.

@TheL0L TheL0L self-assigned this Apr 13, 2026
@content-bot
Copy link
Copy Markdown
Contributor

content-bot commented Apr 13, 2026

🤖 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

@TheL0L TheL0L added python Pull requests that update Python code 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 Apr 13, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 13, 2026 •
edited
Loading

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/Pwned/Integrations/PwnedV2
   PwnedV2.py1953781%48, 54–55, 68–69, 114–117, 125–126, 128, 130, 137, 156, 173–176, 178–181, 183, 185, 206–207, 218–219, 246–247, 249–253, 255
TOTAL1953781% 

Tests Skipped Failures Errors Time
13 0 💤 0 ❌ 0 🔥 2.520s ⏱️

@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 Apr 13, 2026
@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

marketplace-ai-reviewer commented Apr 13, 2026

🤖 Analysis started. Please wait for results...

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

marketplace-ai-reviewer commented Apr 13, 2026

🤖 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 Apr 13, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

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

Choose a reason for hiding this comment

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

🚨 Security Review Required

This PR triggered security alerts that need reviewer attention.

@richardbluestone, @kamalq97, @TheL0L — this review cannot be approved until the security findings are resolved.

Please contact infra team for more details.

github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 13, 2026
View reviewed changes
Comment thread Packs/Pwned/Integrations/PwnedV2/PwnedV2_test.py Dismissed
Comment thread Packs/Pwned/Integrations/PwnedV2/PwnedV2_test.py Dismissed
Comment thread Packs/Pwned/Integrations/PwnedV2/PwnedV2_test.py Dismissed
Comment thread Packs/Pwned/Integrations/PwnedV2/PwnedV2_test.py Dismissed
@richardbluestone richardbluestone requested review from tschanfeld and removed request for richardbluestone April 13, 2026 11:03
@tschanfeld
Copy link
Copy Markdown
Contributor

tschanfeld commented Apr 13, 2026

looks good

kamalq97
kamalq97 requested changes Apr 13, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@kamalq97 kamalq97 left a comment

Choose a reason for hiding this comment

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

Looks good! Let's adjust the context outputs

Comment thread Packs/Pwned/Integrations/PwnedV2/command_examples.txt
Comment thread Packs/Pwned/Integrations/PwnedV2/README.md Outdated
Comment thread Packs/Pwned/Integrations/PwnedV2/README.md Outdated
Comment thread Packs/Pwned/Integrations/PwnedV2/PwnedV2.py
Comment thread Packs/Pwned/Integrations/PwnedV2/PwnedV2.py
@content-bot
Copy link
Copy Markdown
Contributor

content-bot commented Apr 14, 2026

Validate summary
The following errors were thrown as a part of this pr: BA129.
The following errors can be ignored: BA129.
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.
The following errors don't run as part of the nightly flow and therefore can be force merged: BA129.

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

@content-bot
Copy link
Copy Markdown
Contributor

content-bot commented May 15, 2026

This PR is marked as 'Stale' because it has been open for 30 days with no activity, it will be automatically closed in 15 days if no activity will be done. To reset the counter just remove the 'Stale' label or make changes to update this PR. If you wish this PR will never be marked as 'Stale' add the 'Ignore Stale'

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

@kamalq97 kamalq97 kamalq97 requested changes

@tschanfeld tschanfeld Awaiting requested review from tschanfeld

+1 more reviewer

@github-advanced-security github-advanced-security[bot] github-advanced-security[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Requested changes must be addressed to merge this pull request.

Assignees

@TheL0L TheL0L

Labels

docs-approved enhancement python Pull requests that update Python code ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines Stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@TheL0L @content-bot @marketplace-ai-reviewer @tschanfeld @kamalq97 @github-advanced-security