New collector - Menlo Security by RosenbergYehuda · Pull Request #44002 · demisto/content

RosenbergYehuda · 2026-04-23T23:06:24Z

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

In Progress
Ready
In Hold - (Reason for hold)

Related Issues

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

Description

A new collector

Must have

Tests
Documentation

…enlo-Security/CIAC-16573

content-bot · 2026-04-23T23:07:39Z

🤖 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 · 2026-04-23T23:10:29Z

Coverage Report

File	Stmts	Miss	Cover	Missing
Packs/MenloSecurity/Integrations/MenloSecurity
MenloSecurity.py	266	48	81%	78, 80–82, 150, 234–235, 269–270, 300, 370, 417–419, 471–472, 480–481, 516, 518, 555–557, 559–563, 565–566, 571, 573–574, 576–577, 579–582, 589–591, 593–594, 601, 603–605
TOTAL	266	48	81%

Tests	Skipped	Failures	Errors	Time
41	0 💤	0 ❌	0 🔥	9.652s ⏱️

…enlo-Security/CIAC-16573

guardrails · 2026-04-25T21:34:21Z

All previously detected findings have been fixed. Good job! 👍🎉

We will keep this comment up-to-date as you go along and notify you of any security issues that we identify.

👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

…enlo-Security/CIAC-16573

marketplace-ai-reviewer · 2026-04-27T13:10:58Z

🤖 Analysis started. Please wait for results...

…enlo-Security/CIAC-16573

RosenbergYehuda · 2026-05-13T12:54:41Z

@marketplace-ai-reviewer start review

marketplace-ai-reviewer · 2026-05-13T13:01:06Z

🤖 Analysis started. Please wait for results...

marketplace-ai-reviewer · 2026-05-13T13:40:44Z

🤖 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

Thanks for contributing the Menlo Security integration! I've reviewed your PR and found a few areas for improvement, primarily around error logging, YAML configuration, and documentation.

Python Code: Please include full tracebacks in your error logs for better debugging, and avoid excessive debug logging inside loops.
YAML & Metadata: Ensure standard key ordering (e.g., name and display at the top), follow the vendor-entity-action naming convention for commands, and add the missing created key to the pack metadata.
Documentation: Update the description with steps to obtain the API Auth Token, and regenerate the README using the demisto-sdk to fix MDX formatting issues and parameter mismatches.

Let me know if you have any questions!

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

…enlo-Security/CIAC-16573

+        # web: 1 dup removed, 1 new event kept. audit: 1 event (no dedup on first fetch).
+        assert len(events) == 2
+        domains = {e.get("domain") for e in events if "domain" in e}
+        assert "fresh.example.com" in domains


content-bot · 2026-05-14T15:50:32Z

Validate summary
The following errors were reported as warnings: BC117 (Approved).
✅ BC117 validation has been approved by PM review (supported-modules-approved label present).
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? ✅

content-bot · 2026-05-14T16:08:41Z

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: 514d02977e66077d

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

⚠️ AI-generated triage. Validate before acting.

init

2467c2e

RosenbergYehuda self-assigned this Apr 23, 2026

Merge remote-tracking branch 'origin/master' into YR/New-collector--M…

543babf

…enlo-Security/CIAC-16573

RosenbergYehuda added 2 commits April 26, 2026 00:33

multi thread

ae567b1

Merge remote-tracking branch 'origin/master' into YR/New-collector--M…

d419c49

…enlo-Security/CIAC-16573

RosenbergYehuda added 2 commits April 27, 2026 15:55

improvements

4feb36e

Merge remote-tracking branch 'origin/master' into YR/New-collector--M…

a83b33c

…enlo-Security/CIAC-16573

RosenbergYehuda added the docs-approved label Apr 27, 2026

RosenbergYehuda marked this pull request as ready for review April 27, 2026 12:56

RosenbergYehuda added ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines supported-modules-approved ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. labels Apr 27, 2026

RosenbergYehuda requested a review from DanielTal87 April 27, 2026 13:06

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

This comment was marked as outdated.

Sign in to view

This comment has been minimized.

Sign in to view

RosenbergYehuda added 2 commits May 4, 2026 00:00

new docs

703c7ba

Merge remote-tracking branch 'origin/master' into YR/New-collector--M…

9694d43

…enlo-Security/CIAC-16573

This comment has been minimized.

Sign in to view

RosenbergYehuda added 3 commits May 7, 2026 10:00

fix parsing issue

7f8f3a1

ruff

175e406

Merge remote-tracking branch 'origin/master' into YR/New-collector--M…

c0e283e

…enlo-Security/CIAC-16573

This comment has been minimized.

Sign in to view

RosenbergYehuda added 3 commits May 10, 2026 02:14

add logs

ff44252

Merge remote-tracking branch 'origin/master' into YR/New-collector--M…

9d8a00a

…enlo-Security/CIAC-16573

improve resp type

9a1b9bc

RosenbergYehuda added 2 commits May 11, 2026 11:29

fix wrapper issue

628339e

Merge remote-tracking branch 'origin/master' into YR/New-collector--M…

cde4248

…enlo-Security/CIAC-16573

This comment has been minimized.

Sign in to view

RosenbergYehuda added 4 commits May 13, 2026 14:36

changes

870c3a8

Merge remote-tracking branch 'origin/master' into YR/New-collector--M…

4e4cbbc

…enlo-Security/CIAC-16573

enforce

2dfb6ac

polish

df1dc3c

RosenbergYehuda added the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label May 13, 2026

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

marketplace-ai-reviewer reviewed May 13, 2026

View reviewed changes

This comment has been minimized.

Sign in to view

RosenbergYehuda added 3 commits May 13, 2026 17:53

ai review

009f67d

build issues

3da8acd

Merge remote-tracking branch 'origin/master' into YR/New-collector--M…

7aad012

…enlo-Security/CIAC-16573

This comment has been minimized.

Sign in to view

RosenbergYehuda added 4 commits May 13, 2026 23:05

לsha256

1e55ce3

Merge remote-tracking branch 'origin/master' into YR/New-collector--M…

1d75a13

…enlo-Security/CIAC-16573

reduce pagination

6b633c0

revert

8adb52c

This comment has been minimized.

Sign in to view

RosenbergYehuda added 2 commits May 14, 2026 15:41

improve tests

80d699e

Merge remote-tracking branch 'origin/master' into YR/New-collector--M…

9638342

…enlo-Security/CIAC-16573

github-advanced-security AI found potential problems May 14, 2026

View reviewed changes

Comment thread Packs/MenloSecurity/Integrations/MenloSecurity/MenloSecurity_test.py Fixed

Code scanning fix

0fa758d

github-advanced-security AI found potential problems May 14, 2026

View reviewed changes

Comment thread Packs/MenloSecurity/Integrations/MenloSecurity/MenloSecurity_test.py

# web: 1 dup removed, 1 new event kept. audit: 1 event (no dedup on first fetch).

assert len(events) == 2

domains = {e.get("domain") for e in events if "domain" in e}

assert "fresh.example.com" in domains

Conversation

RosenbergYehuda commented Apr 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Contributing to Cortex XSOAR Content

Status

Related Issues

Description

Must have

Uh oh!

content-bot commented Apr 23, 2026

🤖 AI-Powered Code Review Available

Uh oh!

github-actions Bot commented Apr 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

guardrails Bot commented Apr 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

marketplace-ai-reviewer commented Apr 27, 2026

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

RosenbergYehuda commented May 13, 2026

Uh oh!

marketplace-ai-reviewer commented May 13, 2026

Uh oh!

marketplace-ai-reviewer commented May 13, 2026

Uh oh!

marketplace-ai-reviewer left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

Uh oh!

content-bot commented May 14, 2026

Uh oh!

content-bot commented May 14, 2026

🔍 AI Triage Report Available

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

RosenbergYehuda commented Apr 23, 2026 •

edited

Loading

github-actions Bot commented Apr 23, 2026 •

edited

Loading

guardrails Bot commented Apr 25, 2026 •

edited

Loading