Skip to content

Remove wireshark#44278

Open
BEAdi wants to merge 7 commits into
masterfrom
ae-pcap-20260512
Open

Remove wireshark#44278
BEAdi wants to merge 7 commits into
masterfrom
ae-pcap-20260512

Conversation

@BEAdi
Copy link
Copy Markdown
Contributor

@BEAdi BEAdi commented May 12, 2026

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

Related: CIAC-16766

Description

Must have

  • Tests
  • Documentation

@content-bot
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

@BEAdi BEAdi added the ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines label May 12, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 12, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/CommonScripts/Scripts/PcapHTTPExtractor
   PcapHTTPExtractor.py16210932%64, 76–81, 83–85, 87–88, 96, 98, 111–112, 116, 121, 134–136, 139–140, 142, 152–153, 163, 167–170, 181, 184–191, 193–195, 197–198, 200, 214, 217, 219–221, 223–224, 227–229, 231–232, 242, 260–264, 266, 282–283, 286–287, 290, 293, 296–297, 300, 302–304, 306–311, 314–317, 319, 330, 332–334, 336, 338, 348–349, 351–352, 356, 358, 381, 391–394, 397–399, 402
Packs/PcapAnalysis/Scripts/PcapConvert
   PcapConvert.py1082279%21, 29–31, 52, 70–73, 75–76, 88, 104–105, 109, 118, 124, 137–139, 143, 146
Packs/PcapAnalysis/Scripts/PcapExtractStreams
   PcapExtractStreams.py24320416%38, 43–45, 60, 102–103, 105–112, 114–120, 122–127, 129–130, 132, 144–145, 147–154, 156–157, 167–170, 184–186, 200, 205, 214–218, 220–221, 223–224, 226–227, 237–238, 240–241, 248, 261–263, 266–270, 278–280, 285, 297–300, 302–306, 308, 310, 321–324, 326–331, 334–340, 343–346, 349–350, 352–353, 355–356, 359, 370–381, 383, 385–391, 393, 408–415, 424–426, 428–430, 432–434, 440–447, 449, 460–464, 466, 468, 470–471, 474, 483–485, 487, 494–503, 505–511, 513, 515, 526–528, 530, 532
Packs/PcapAnalysis/Scripts/PcapFileExtractStreams
   PcapFileExtractStreams.py1488741%25, 28, 32–33, 35, 57, 99–100, 102–109, 111–117, 119–124, 126–127, 129, 141–142, 144–151, 153–154, 164–167, 181–183, 197, 213–215, 217–218, 220–221, 223–224, 234–235, 237–238, 245, 260, 277, 300, 304, 308, 317–318, 321, 331–332, 334–335, 337, 343, 348, 353, 366–367, 369–370, 376
Packs/PcapAnalysis/Scripts/PcapFileExtractor
   PcapFileExtractor.py873658%26–29, 41, 53, 65, 77–78, 93–95, 203–204, 206–208, 210–213, 215–217, 219, 230, 232, 235–236, 262–266, 278–279
Packs/PcapAnalysis/Scripts/PcapMinerV2
   PcapMinerV2.py42326736%80–82, 90, 93, 97, 101–102, 104, 110, 112, 114–115, 121–122, 129, 133–135, 146–152, 156–160, 168, 172–173, 180, 184–185, 193, 197, 199–200, 202, 204–207, 211–213, 223, 227–230, 237, 241–244, 248–251, 253–254, 256–258, 262–269, 278–283, 291, 295–297, 304, 307, 309, 321–325, 327–328, 330–331, 333–334, 336–337, 339–340, 342–343, 345–346, 348–349, 351–352, 354–355, 357–358, 360–361, 363–365, 367–368, 372–374, 380–386, 389, 400–409, 415–417, 422–434, 436, 438–440, 469, 480–481, 483, 485–487, 490–492, 495, 498, 501–507, 509–514, 517–522, 525–531, 542–546, 549–554, 571–572, 583–587, 589, 592, 598, 680–686, 699–702, 714–715, 760–762, 764–779, 781–783, 785–789, 791–792, 794–795
TOTAL117172538% 

Tests Skipped Failures Errors Time
13 0 💤 2 ❌ 1 🔥 6.319s ⏱️

@content-bot

This comment has been minimized.

@content-bot
Copy link
Copy Markdown
Contributor

This PR was automatically updated by a GitHub Action

  • CommonScripts pack version was bumped to 1.22.12.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@content-bot

This comment has been minimized.

@content-bot
Copy link
Copy Markdown
Contributor

This PR was automatically updated by a GitHub Action

  • CommonScripts pack version was bumped to 1.22.13.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@content-bot
Copy link
Copy Markdown
Contributor

This PR was automatically updated by a GitHub Action

  • CommonScripts pack version was bumped to 1.22.14.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@content-bot
Copy link
Copy Markdown
Contributor

Validate summary
The following errors were thrown as a part of this pr: DO101.
The following errors cannot be ignored: DO101.
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
Copy link
Copy Markdown
Contributor

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: b520b46efc9a7446

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

⚠️ AI-generated triage. Validate before acting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants