Skip to content

Nbensalmon/ciac 14500 tpb file enrichment #41214

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 37 commits into from
Sep 14, 2025
Merged

Nbensalmon/ciac 14500 tpb file enrichment #41214

merged 37 commits into from
Sep 14, 2025
+1,823 −2

Conversation

nbensalm-palo
Copy link
Contributor

Related Issues

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

Description

Added a Test playbook for File Enrichment Script.

nbensalm-palo and others added 22 commits August 26, 2025 17:21
@nbensalm-palo
Resolve bug CIAC-14559
bd3308f
@nbensalm-palo
Added test for CIAC-14559
ec314c3
@nbensalm-palo
First Implementation internal + external
921436a
@nbensalm-palo
Add start of Internal and external
0bff294
@nbensalm-palo
Merge branch 'master' into nbensalmon/CIAC-14500_TPB-FileEnrichment
d5eecfb
@nbensalm-palo
Adding internal and external inculde tim
3d6b7dc
@nbensalm-palo
Adding internal and external test combined
acf0432
@nbensalm-palo
Start implementing sad Flows
4515b03
@nbensalm-palo
Added Error path and change task name
92a86d9
@nbensalm-palo
Adding test case
33045c1
@nbensalm-palo
Change name
807ac8b
@nbensalm-palo
Change command to normal
fd62aa6
@nbensalm-palo
Update name and conf.json
3094a97
@nbensalm-palo
Ruff format
7900c4e
@nbensalm-palo
Update release notes
53d2a40
@nbensalm-palo
Merge branch 'master' into nbensalmon/CIAC-14500_TPB-FileEnrichment
b1e09f9
@nbensalm-palo
Updated changes after review
7c97aad
Merged master into current branch.
bcc15ed
Bump pack from version AggregatedScripts to 1.1.6.
b3aefae
@nbensalm-palo
Merge branch 'master' into nbensalmon/CIAC-14500_TPB-FileEnrichment
29318bc
@nbensalm-palo
Adapting Playbook to fit XSOAR
208a899
@nbensalm-palo
Update release notes
8199dbe
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 8, 2025 •
edited
Loading

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/AggregatedScripts/Scripts/FileEnrichment
   FileEnrichment.py3214386%121–122, 146–147, 187–190, 199–200, 207, 227, 237, 375, 543–544, 586–587, 590, 593–594, 598–601, 692, 709, 741, 801–803, 805–806, 808–809, 811–812, 877, 904, 906, 911, 916–917
TOTAL3214386% 

Tests Skipped Failures Errors Time
33 0 💤 0 ❌ 0 🔥 2.303s ⏱️

@nbensalm-palo nbensalm-palo removed the request for review from JudahSchwartz September 8, 2025 13:51
@content-bot
Copy link
Collaborator

This PR was automatically updated by a GitHub Action

  • AggregatedScripts pack version was bumped to 1.1.11.

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

@nbensalm-palo nbensalm-palo added the ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines label Sep 8, 2025
Shellyber
Shellyber reviewed Sep 9, 2025
View reviewed changes
Copy link
Contributor

@Shellyber Shellyber left a comment

Choose a reason for hiding this comment

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

nice

@content-bot
Copy link
Collaborator

This PR was automatically updated by a GitHub Action

  • AggregatedScripts pack version was bumped to 1.1.12.

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

@content-bot
Copy link
Collaborator

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

@content-bot
Copy link
Collaborator

This PR was automatically updated by a GitHub Action

  • AggregatedScripts pack version was bumped to 1.1.13.

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

Shellyber
Shellyber approved these changes Sep 14, 2025
View reviewed changes
Copy link
Contributor

@Shellyber Shellyber 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

@nbensalm-palo nbensalm-palo merged commit b48806b into master Sep 14, 2025
23 of 24 checks passed
@nbensalm-palo nbensalm-palo deleted the nbensalmon/CIAC-14500_TPB-FileEnrichment branch September 14, 2025 11:11
scohenkadosh pushed a commit that referenced this pull request Sep 15, 2025
@nbensalm-palo @scohenkadosh
Nbensalmon/ciac 14500 tpb file enrichment (#41214)
38e27f2 
Added a Test playbook for File Enrichment Script.
xsoar-bot pushed a commit to xsoar-contrib/content that referenced this pull request Sep 17, 2025
@nbensalm-palo @xsoar-bot
Nbensalmon/ciac 14500 tpb file enrichment (demisto#41214)
e10f00a 
Added a Test playbook for File Enrichment Script.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shellyber Shellyber Shellyber approved these changes

Assignees
No one assigned
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.
3 participants
@nbensalm-palo @content-bot @Shellyber