forked from Azure/Azure-Sentinel
-
Notifications
You must be signed in to change notification settings - Fork 0
TEST: Automation safety check - DO NOT MERGE #3
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.
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
Closed
Closed
Changes from all commits
Commits
Show all changes
1526 commits
Select commit
Hold shift + click to select a range
5b7e35f
Refresh for endace-usa
peter-endace 4abb7cc
Refresh for endace-usa
peter-endace 86801f9
Refresh for endace-usa
peter-endace e74da66
Merge pull request #13414 from Azure/v-shukore/sentinelOne
v-dvedak 64f82f2
chore: Update Solutions Analyzer CSV files and documentation (#6)
github-actions[bot] a22760a
chore: Update Solutions Analyzer CSV files (#1)
github-actions[bot] 1bb2d1a
Update Cyble Vision connector and playbook configs
prasad-cyble 82510a3
Merge pull request #13385 from Azure/dependabot/pip/Solutions/ZeroFox…
v-atulyadav 45b1a95
Merge pull request #13386 from Azure/dependabot/pip/Solutions/Rapid7I…
v-atulyadav 6705f2c
Bump urllib3 in /Solutions/WithSecureElementsViaFunction/Data Connectors
dependabot[bot] dae20d5
Merge pull request #13383 from kingwil/wiking-sap-validations
v-atulyadav 0cdd8b0
Merge branch 'master' into dependabot/pip/Solutions/ESET-Protect-Plat…
v-atulyadav 036939a
Update ReleaseNotes.md
prasad-cyble fb14516
Merge pull request #13404 from Azure/dependabot/pip/Solutions/ESET-Pr…
v-atulyadav ae0826e
Update IncidentOverview.json
v-sabiraj b8fa131
Add exception for CodeQL issue
oshezaf f9b897c
Update ExchangeCompromiseHunting.json
v-sabiraj d387204
Update GainCodeExecutionADFSviaWMI.yaml
v-sabiraj 9828a9b
Update AWS solution to version 3.0.8 and improve descriptions
v-sabiraj 6edb881
Update Network Threat Protection Essentials to v3.0.2
v-sabiraj 54c6cc2
Bump Okta SSO solution to version 3.1.4
v-sabiraj 41876ca
Updated for pagination issue
v-utpalkumar 2a0819e
Update PaloAlto-PAN-OS to 3.0.11 and Okta SSO rule
v-sabiraj 36f57f7
Zip updated for GetInventory
v-utpalkumar d73cc8c
Bump Windows Security Events solution to v3.0.10
v-shukore da2f5d8
Bump AWSAthena solution to 3.0.2, update Python version
v-kasghosh 679ee87
Updated for pagination
v-utpalkumar 19fa8bf
Zip updated for pagination
v-utpalkumar b1c8c51
Merge pull request #13149 from Azure/dependabot/npm_and_yarn/multi-75…
v-dvedak 3476a7a
Update Zscaler ZPA Unexpected Session Duration rule to use ConnectionID
v-kasghosh 768a423
Merge pull request #13372 from andrewj-t/fix/NonDCActiveDirectoryRepl…
v-atulyadav 196b10e
Update ReleaseNotes.md
v-kasghosh 4b6e28e
Updated to count how many times the while loop runs
v-utpalkumar 9c6fef8
Zip updated
v-utpalkumar 3e9f76e
Update Cyble Vision API endpoints and payloads
prasad-cyble 65a0f6a
fix: Reset WorkbooksMetadata.json to upstream format, add only Cyren …
mazamizo21 f44bfb9
Bump up Python version to 3.12
shivangdpatel 984b789
[ASIM] Make changes to Authentication ASIM OktaSSO and OktaV2 parser …
yummyblabla b1e727b
Revise Power BI connector setup instructions
Justfyn d938202
Updated package file
peter-endace 2dbc541
Merge pull request #13195 from Azure/v-sudkharat/Updating-ExtensionBu…
v-atulyadav 5e915f8
Merge pull request #13425 from Azure/v-kasghosh/issue_number/12588
v-atulyadav 2d2a655
Update Cyble Vision package to new version
v-maheshbh 9371f63
Add UEBA Behaviors Layer documentation and reference materials (#13426)
shlomimaa10-art 4cdd69f
Release RiskIQ Solution v3.0.0 with ARM template updates
v-sabiraj 5286db0
Merge pull request #13382 from prasad-cyble/Cyble/pollingupdate
v-atulyadav ef0dfa7
Merge pull request #13374 from kingwil/createsolution-update
v-atulyadav 6fe1479
Update CMMC 2.0 solution to version 3.1.1
v-sabiraj c1f3ec3
Update release notes with non-functional link changes
v-sabiraj 255f7f1
Merge branch 'master' into v-sabiraj-updatingbrokenlinks
v-sabiraj 1bfec89
Update CMMC 2.0 package and refine template mappings
v-sabiraj 9189878
updated the exclusion list
Alekhya0824 8a9ec81
Merge branch 'master' into AwsCloudTrailParser
Alekhya0824 e18e81a
Merge pull request #13405 from Azure/v-sabiraj-updatingbrokenlinks
v-atulyadav 3e51cd5
Merge pull request #13396 from recordedfuture/fix-correct-deploy-link
v-atulyadav dc18e7e
Revert "chore: Update Solutions Analyzer CSV files and documentation …
RamboV cf3bd5c
Readme file updated
v-utpalkumar aef9caa
Updated the logic app
v-utpalkumar 9fe8e13
fix: Add missing logoFileName to CloudflareCCFWorkbook entry
mazamizo21 0e708b6
fix: Remove spaces in Logo img tag to match Cloudflare pattern
mazamizo21 0f4b5e5
fix: Remove logoFileName fields and packageMetadata.json to match Clo…
mazamizo21 077b07f
fix: Use standard Azure Sentinel logo path instead of custom logo
mazamizo21 e2e60a4
fix: Add logoFileName field to workbook entries to pass validation
mazamizo21 6d84972
feat: Add Cyren logo to Logos/ directory and update Solution_Cyren.json
mazamizo21 f879d3c
Fix logo structure and format for TacitRed Defender TI solution
mazamizo21 f41f716
Update bloodhound function app to use vanity name for WEBSITE_RUN_FRO…
7da513f
Update IncidentOverview.json
v-sabiraj 8759fc1
Deprecate IncidentOverview workbook template
v-sabiraj bd1ac18
Update SOC Handbook solution to version 3.0.6
v-sabiraj 231d650
Remove Incident Overview workbook from solution
v-sabiraj 42298fa
Merge branch 'master' into AwsCloudTrailParser
v-sabiraj 4e181eb
validations
v-sabiraj 1b9d904
Updated WEBSITE_RUN_FROM_PACKAGE to use Microsoft-managed aka.ms URL
4f8f712
updated zip to have the latest mainTemplate
jamos-bt b66f859
Merge branch 'master' of https://github.com/Azure/Azure-Sentinel
jamos-bt ba92205
corrected version to 1.0.0 since it's initial release
jamos-bt 412d384
Update LastUpdated date in AWS CloudTrail parsers
v-sabiraj 8618985
BigID DSPM Asset expanding and mapping
mhebrard-bigid 266bb21
Add workbook preview images to solution folder
mazamizo21 f1f26b7
back to 3.0.0 for the version
jamos-bt 7e4c9ee
[ASIM] ProcessEvent - Microsoft365D Parser changes (#13411)
yummyblabla 49825be
Bump azure-core in /Solutions/SailPointIdentityNow/Data Connectors
dependabot[bot] 556f54c
Bump azure-core
dependabot[bot] 1d0d013
Bump azure-core in /Solutions/ESET Protect Platform/Data Connectors
dependabot[bot] 7720189
Remove Cloudflare contamination from WorkbooksMetadata.json
mazamizo21 03fb82c
Remove WorkbooksMetadata.json contamination
mazamizo21 956400a
Merge pull request #13421 from peter-endace/master
v-atulyadav adfe474
Merge pull request #13417 from Justfyn/MDO_v3
v-atulyadav 9f8249f
Merge pull request #13407 from mhebrard-bigid/bigid-asset-expand-and-…
v-atulyadav e93f786
Merge branch 'master' into AwsCloudTrailParser
v-sabiraj 18e8b50
validations
v-sabiraj a679485
Updated the zip
v-kasghosh 4e005ee
Merge pull request #13408 from Azure/v-sabiraj-updatingincidentovervi…
v-dvedak 4984fc9
Merge branch 'master' into dependabot/pip/Solutions/WithSecureElement…
v-atulyadav df31ff4
update the zip
v-kasghosh e395c29
Fix Validations
v-kasghosh 683b89b
Merge pull request #13422 from Azure/dependabot/pip/Solutions/WithSec…
v-atulyadav 2e0859f
Merge branch 'master' into dependabot/pip/Solutions/SailPointIdentity…
v-atulyadav 12b8459
Updated release notes
v-utpalkumar 3bfda12
Merge pull request #13435 from Azure/dependabot/pip/Solutions/SailPoi…
v-atulyadav 7c460c6
Merge branch 'master' into dependabot/pip/Solutions/Lookout-Cloud-Sec…
v-atulyadav e5fc6ef
Revert "chore: Update Solutions Analyzer CSV files (#1)"
941ed26
Update Playbook_ARM_Template_Generator.zip
v-kasghosh 802a056
Removed few logging statements
v-utpalkumar 5f3712b
Merge pull request #13424 from Azure/v-kasghosh/issue_number/13412
v-atulyadav 4dc9c5a
Updated logging info
v-utpalkumar 439bf5e
adds default logo
1f099af
Merge pull request #13436 from Azure/dependabot/pip/Solutions/Lookout…
v-atulyadav ac786cf
Merge pull request #13439 from Azure/v-kasghosh/issue_number/13163
v-dvedak 673e4d8
Merge branch 'master' into pr/13397
v-shukore e079ca0
chore: Update Solutions Analyzer CSV files (#2)
github-actions[bot] 2c59b20
Revert "chore: Update Solutions Analyzer CSV files (#2)"
v-shukore fbc3aec
Merge pull request #13392 from Azure/v-kasghosh/issue_number/12271
v-utpalkumar d99938f
adds logo
0d1c972
Merge branch 'master' into AwsCloudTrailParser
Alekhya0824 bd8fc4f
fix: Update Cyren workbook logoFileName to cyren_logo.svg and remove …
mazamizo21 e0cb2d5
Bump urllib3 from 2.6.0 to 2.6.3 in /Solutions/Box/Data Connectors (#…
dependabot[bot] 8c262a5
feat(Cyren): Add ConnectionToggleButton and textbox inputs to connect…
mazamizo21 cabc45c
Bump azure-core in /Solutions/ESET Protect Platform/Data Connectors
dependabot[bot] 88c3b23
fix: Restore Logos/cyren_logo.svg
mazamizo21 30df58e
Merge branch 'master' into dependabot/pip/Solutions/ESET-Protect-Plat…
v-atulyadav 3b4f4ce
fix(Cyren): V3 repackage with ConnectionToggleButton and post-V3 fixes
mazamizo21 f131641
Merge branch 'dependabot/pip/Solutions/ESET-Protect-Platform/Data-Con…
v-atulyadav 9153810
fix(Cyren): Correct API versions to 2025-09-01 stable (not preview)
mazamizo21 7f3497e
fix: Add missing logoFileName to CloudflareCCFWorkbook (upstream bug …
mazamizo21 01c5d00
fix: Remove duplicate cyren_logo.svg from Workbooks/Images/Logos - on…
mazamizo21 1dbb4c2
fix: Restore cyren_logo.svg to Workbooks/Images/Logos/ (matches Cloud…
mazamizo21 b0d4a81
fix(Cyren): V3 repackage with stable API versions (2025-09-01)
mazamizo21 4b11bfe
fix: table retention minimum 4 days (Azure rejects 1 day)
d0c8a75
fix: add Bearer prefix and fix parameter names for JWT auth
21358ed
fix: use ApiKeyIdentifier for Bearer prefix per MS CCF docs
a418b0d
fix: correct ARM template bracket syntax for parameters (single ] not…
ca97c82
Merge pull request #13430 from keith-degrace/bloodhound
hassanchawiche 493d7ea
Merge pull request #13297 from Azure/update-asim-tester-csv
oshezaf 3903a64
[ASIM] Authentication AADSigninLogs parser rewrite (#13409)
yummyblabla 0c25fe6
Update WorkbooksMetadata.json
v-shukore 6470305
Merge pull request #13397 from secpfe/master
v-atulyadav 4ef53a3
Revert "fix: Add missing logoFileName to CloudflareCCFWorkbook (upstr…
v-maheshbh f874d56
Merge branch 'master' into pr/13278
v-maheshbh bca10bc
Add GCP security analytic rules and hunting queries
rahul0216 37073dd
Update GCPDNSSECDisabled.yaml
rahul0216 6057df6
Merge pull request #13427 from leeninc/master
v-atulyadav 51beac3
Merge pull request #13437 from Azure/dependabot/pip/Solutions/ESET-Pr…
v-atulyadav 14cf7ff
Update ValidConnectorIds.json
rahul0216 23a39d7
Bump azure-core in /Solutions/IllumioSaaS/Data Connectors
dependabot[bot] 3746977
Remove DNSSEC Disabled analytic rule
rahul0216 f73c95a
Delete GCPDNSSECDisabled.yaml
rahul0216 383b132
Merge pull request #13244 from ryantully-archtis/archtis-ncprotect-3-…
v-dvedak 4548d2e
docs: Add workbook testing document for Cyren Threat Intelligence
mazamizo21 b6adca8
Merge branch 'master' into pr/13278
v-maheshbh fce9085
Added GCPDNSSECDisabled.yaml again
rahul0216 c56e7dc
Remove unused ResourceName mapping in GCP VM snapshot rule
rahul0216 7df146d
update halcyon solution's publisherId
444d9eb
Merge pull request #13394 from Azure/tools/map-connectors-to-tables
oshezaf 09389bc
Renamed analytic rule GCPDNSSECDisabledForDNSZone
rahul0216 dd12798
Updated to 7.2
oshezaf 62fb308
Merge branch 'master' into tools/map-connectors-to-tables
oshezaf 877138a
Update Hunting queries'names
rahul0216 5102166
Set fetch-depth to 2 in workflow checkouts (#13450)
rahul0216 2acdec5
Merge pull request #13278 from Data443/feature/cyren-threat-intelligence
v-dvedak 5436d12
fixed is_deprecated
oshezaf c901e63
Merge branch 'master' into tools/map-connectors-to-tables
oshezaf 8fbcbbd
Merge pull request #13451 from Halcyon-Sentinel-Integration/master
v-atulyadav 70f066f
Merge branch 'master' into pr/13074
v-maheshbh cb7ff52
Merge pull request #13074 from jamos-bt/master
v-dvedak 4bb1616
Merge branch 'master' into pr/13359
v-atulyadav dd70d91
Merge branch 'master' into dependabot/pip/Solutions/IllumioSaaS/Data-…
v-atulyadav b81e741
Merge branch 'master' into pr/13266
v-maheshbh 47c5405
Merge pull request #13447 from Azure/dependabot/pip/Solutions/Illumio…
v-atulyadav 262eed4
Merge pull request #13359 from anthonylamark/SquadraTechnologiesSecRm…
v-atulyadav fc5fe84
Add files via upload
RamboV 67469fa
update readme.md
RamboV 3afc14f
Add files via upload
RamboV 80f9823
Update readme
RamboV 94af79d
fix(TacitRed-Defender-TI): Replace reference() with FunctionAppUrl pa…
mazamizo21 efad58c
Update ASimTester.csv (#13461)
yummyblabla 4f1b9c0
Add support for standalone content items
oshezaf feb62fb
Merge pull request #13463 from Azure/tools/map-connectors-to-tables
oshezaf 17ca902
updated url for ioc connector
chkp-jkohn 52b29f4
removed duplicate 'schema' nesting, connector working
chkp-jkohn 892c9de
chore: Update Solutions Analyzer CSV files and documentation (#7)
github-actions[bot] e5fa61d
Merge branch 'master' into users/rahul/gcpauditlogs-content
rahul0216 175347e
Update ZeroTrust(TIC3.0) to version 3.0.3
v-shukore 0a3b921
Bump ZeroTrust solution to version 3.0.3
v-shukore 816d29c
Release NISTSP80053 solution version 3.0.3
v-shukore 1a925fd
Update ReleaseNotes.md
v-shukore b2f2883
Update Solution_GCPAuditLogs.json
v-shukore c03c46d
Revert "chore: Update Solutions Analyzer CSV files and documentation …
v-maheshbh 68e621b
Remove trailing blank line from mainTemplate.json
v-maheshbh a696075
chore: Update Solutions Analyzer CSV files and documentation (#8)
github-actions[bot] 91ad82e
Merge pull request #13446 from Azure/users/rahul/gcpauditlogs-content
v-atulyadav f4ce16b
fix: Repackage solution using V3 tooling (no manual mainTemplate edits)
mazamizo21 96fa215
ok
mazamizo21 f8d6bf7
[ASIM] Authentication - Crowdstrike FalconHost Parser changes (#13462)
yummyblabla e0551de
Filter on TimeGenerated (#13467)
yummyblabla f14fcad
Revert "chore: Update Solutions Analyzer CSV files and documentation …
v-maheshbh 0559389
Merge pull request #12671 from Azure/AwsCloudTrailParser
v-atulyadav b4a8e7f
Add Conditional Access SISM workbook to Entra ID solution
v-maheshbh ee79516
Merge pull request #13468 from Azure/v-maheshbh/Microsoft-Entra-ID
v-atulyadav 39fa412
fix packaging (#13476)
AmirSasson 56f1e12
[ASIM] NetworkSession - NTANetAnalytics - SrcIpAddr can map to SrcIp …
yummyblabla dc6bfac
[ASIM] Authentication - Sshd Parser fixes (#13460)
yummyblabla 32c5154
Merge pull request #13341 from chrisda/EOP-chrisda
v-dvedak 20f94a3
Update JudgementPandaExfilActivity.yaml
v-sabiraj 27c71a9
Update SUNSPOTHashes.yaml
v-sabiraj 451587e
Update LocalDeviceJoinInfoAndTransportKeyRegKeysAccess.yaml
v-sabiraj 1fd3bb3
Update MITRE technique and add reference link
v-sabiraj 5c52ae9
fix: Add metadata to Function App azuredeploy.json for Content Hub vi…
mazamizo21 001f62b
Merge pull request #13480 from Azure/v-sabiraj-updatinglinksforrules
v-sabiraj 1a38f46
Update Teams hunting queries to version 1.0.0
v-sabiraj 2f2dea3
repackage
chkp-jkohn e7a0ed6
Add files via upload
RamboV 6b34330
update readme.md for JoeSandbox URL Analysis Playbook
RamboV 9c0913e
Add files via upload
RamboV 8883b92
Revise readme.md for JoeSandbox File Analysis Playbook
RamboV b790d3c
Update mainTemplate.json and package for XDR solution
v-shukore e538467
Fix syntax in mainTemplate.json parameters
v-shukore 5a70666
Update ReleaseNotes.md
v-shukore 73d79d1
Merge branch 'master' into v-shukore/MS-Defender-XDR
v-shukore a5622a2
Update ReleaseNotes.md
v-shukore 6a7dbce
Merge pull request #13464 from chkp-jkohn/connector-fixes
v-atulyadav 2b573d2
Merge pull request #13205 from Azure/v-shukore/MS-Defender-XDR
v-atulyadav 8224700
Update BasePath in Solution_JoeSandbox.json
v-maheshbh d04dc50
chore: Update Solutions Analyzer CSV files and documentation (#9)
github-actions[bot] 7dcb5fd
Revert "chore: Update Solutions Analyzer CSV files and documentation …
v-maheshbh 24fc642
Update SUNSPOT and Sigma rule references in XDR package
v-sabiraj 750e74c
Merge pull request #12801 from RamboV/master
v-dvedak 9be8a9a
Update 3.0.13.zip
v-sabiraj 157cb5f
Update MITRE technique and fix SUNSPOT rule descriptions
v-sabiraj a00dedb
Updated solution description by eliminating legacy streaming text
v-rusraut 248af01
Update SUNSPOTHashes.yaml
v-shukore 0229a92
Merge pull request #13485 from Azure/v-sabiraj-updatingMicrosoftXDRules
v-atulyadav ea02cfd
Merge pull request #13488 from Azure/v-rusraut/AzureDevOps-Issue13179
v-atulyadav 98d7653
Rename BeyondTrustPMCloud preview images
v-maheshbh 74a3348
Add BeyondTrust logo SVG asset
v-maheshbh 59f1278
fix(TacitRed-Defender-ThreatIntelligence): Add metadata to Function A…
mazamizo21 ed5623e
Merge pull request #13489 from Azure/v-maheshbh/BeyondTrustPMCloud
v-atulyadav ae2591d
fix: Remove duplicate zip file contamination
mazamizo21 4fcfd96
Merge remote-tracking branch 'upstream/master' into feature/tacitred-…
mazamizo21 4c731dd
fix(TacitRed-Defender): Restructure playbook folders for Content Hub …
mazamizo21 23c89ad
fix(TacitRed-Defender-ThreatIntelligence): Update metadata to match r…
mazamizo21 5fe1493
fix(TacitRed-Defender-ThreatIntelligence): Mirror SentinelOne playboo…
mazamizo21 a067b88
fix(TacitRed-Defender-ThreatIntelligence): Fix parentId bracket syntax
mazamizo21 296a219
fix(TacitRed-Defender-TI): Change API versions to match working Senti…
mazamizo21 2f87d36
fix(TacitRed-Defender-TI): Fix bracket escaping in nested metadata re…
mazamizo21 25a1957
fix(TacitRed-Defender-TI): Revert to single brackets in metadata reso…
mazamizo21 8859e1a
fix(TacitRed-Defender-TI): Match exact SentinelOne bracket pattern
mazamizo21 cfea03d
fix(TacitRed-Defender-TI): Apply V3 tooling regenerated mainTemplate
mazamizo21 54edd8a
fix(TacitRed-Defender-TI): Remove reference() from playbook - fixes C…
mazamizo21 44ac7a9
Test: Automation safety check - safe test PR
mazamizo21 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,106 @@ | ||
| name: AWS-S3 DataConnector Bundle Auto-Update | ||
| run-name: Auto-updating AWS-S3 bundles for ${{ github.event.pull_request.head.ref }} | ||
|
|
||
| on: | ||
| pull_request: | ||
| branches: | ||
| - master | ||
| paths: | ||
| # Trigger when any of these files in AWS-S3 directory change | ||
| - 'DataConnectors/AWS-S3/*.ps1' | ||
| - 'DataConnectors/AWS-S3/*.py' | ||
| - 'DataConnectors/AWS-S3/*.md' | ||
| - 'DataConnectors/AWS-S3/CloudFormation/**' | ||
| - 'DataConnectors/AWS-S3/Enviornment/**' | ||
| - 'DataConnectors/AWS-S3/Utils/**' | ||
| # Don't trigger on zip file changes (to avoid recursion) | ||
| - '!DataConnectors/AWS-S3/*.zip' | ||
| # Don't trigger on bundle automation documentation changes (not bundled) | ||
| - '!DataConnectors/AWS-S3/BUNDLE_AUTOMATION.md' | ||
|
|
||
| # Allow manual workflow dispatch for testing | ||
| workflow_dispatch: | ||
|
|
||
| jobs: | ||
| auto-update-bundles: | ||
| # Security: Block workflow execution on forked repositories | ||
| if: ${{ !github.event.pull_request.head.repo.fork }} | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: write | ||
| pull-requests: write | ||
|
|
||
| steps: | ||
| - name: Generate a token | ||
| id: generate_token | ||
| uses: actions/create-github-app-token@v1 | ||
| with: | ||
| app-id: ${{ secrets.APPLICATION_ID }} | ||
| private-key: ${{ secrets.APPLICATION_PRIVATE_KEY }} | ||
|
|
||
| - name: Checkout PR branch with sparse checkout | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| token: ${{ steps.generate_token.outputs.token }} | ||
| ref: ${{ github.event.pull_request.head.ref }} | ||
| fetch-depth: 2 # Just need HEAD and parent for git diff | ||
| persist-credentials: false # Security: Don't persist credentials after checkout | ||
| sparse-checkout: | | ||
| DataConnectors/AWS-S3 | ||
| .script | ||
| sparse-checkout-cone-mode: false | ||
|
|
||
| - name: Restore bundling script from base branch | ||
| run: | | ||
| # Security: Use trusted script from base branch to prevent malicious PR modifications | ||
| # Fetch the base branch to ensure we have the reference | ||
| git fetch origin ${{ github.base_ref || 'master' }}:refs/remotes/origin/${{ github.base_ref || 'master' }} | ||
| git checkout origin/${{ github.base_ref || 'master' }} -- .script/bundleAwsS3Scripts.sh | ||
| chmod +x .script/bundleAwsS3Scripts.sh | ||
|
|
||
| - name: Check if auto-update needed | ||
| id: check_update | ||
| run: | | ||
| # Skip if this commit already updated bundles (prevent loops) | ||
| if git log -1 --name-only | grep -q "ConfigAwsS3DataConnectorScripts.*\.zip"; then | ||
| echo "skip=true" >> $GITHUB_OUTPUT | ||
| echo "Bundles already updated in latest commit" | ||
| else | ||
| echo "skip=false" >> $GITHUB_OUTPUT | ||
| fi | ||
|
|
||
| - name: Update bundles | ||
| if: steps.check_update.outputs.skip != 'true' | ||
| run: | | ||
| .script/bundleAwsS3Scripts.sh | ||
|
|
||
| - name: Commit updated bundles | ||
| if: steps.check_update.outputs.skip != 'true' | ||
| env: | ||
| GITHUB_TOKEN: ${{ steps.generate_token.outputs.token }} | ||
| run: | | ||
| git config --local user.email "action@github.com" | ||
| git config --local user.name "GitHub Action" | ||
|
|
||
| # Configure remote with token for push (needed due to persist-credentials: false) | ||
| git remote set-url origin https://x-access-token:${GITHUB_TOKEN}@github.com/${{ github.repository }}.git | ||
|
|
||
| # Stage zip files | ||
| git add DataConnectors/AWS-S3/ConfigAwsS3DataConnectorScripts*.zip | ||
|
|
||
| # Check if there are changes to commit | ||
| if ! git diff --cached --quiet; then | ||
| git commit -m "Auto-update AWS-S3 DataConnector bundles | ||
|
|
||
| - Updated ConfigAwsS3DataConnectorScripts.zip | ||
| - Updated ConfigAwsS3DataConnectorScriptsGov.zip | ||
| - Changes triggered by: ${{ github.event.pull_request.head.sha }} | ||
|
|
||
| [skip ci]" | ||
|
|
||
| git push origin ${{ github.event.pull_request.head.ref }} | ||
|
|
||
| echo "✅ Successfully updated and committed bundle files" | ||
| else | ||
| echo "ℹ️ No bundle changes detected" | ||
| fi | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,109 @@ | ||
| name: Update Solutions Analyzer CSV Files | ||
|
|
||
| on: | ||
| push: | ||
| branches: | ||
| - master | ||
| paths: | ||
| - 'Solutions/**/*.json' | ||
| - 'Solutions/**/Parsers/**/*.yaml' | ||
| - 'Solutions/**/Parsers/**/*.yml' | ||
| - 'Tools/Solutions Analyzer/map_solutions_connectors_tables.py' | ||
| workflow_dispatch: # Allow manual trigger | ||
| schedule: | ||
| # Run weekly on Monday at 2 AM UTC to catch any missed changes | ||
| - cron: '0 2 * * 1' | ||
|
|
||
| jobs: | ||
| update-csv: | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: write | ||
| pull-requests: write | ||
|
|
||
| steps: | ||
| - name: Checkout repository | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - name: Set up Python | ||
| uses: actions/setup-python@v5 | ||
| with: | ||
| python-version: '3.11' | ||
|
|
||
| - name: Install dependencies | ||
| run: | | ||
| pip install json5 | ||
|
|
||
| - name: Run Solutions Analyzer | ||
| run: | | ||
| cd "Tools/Solutions Analyzer" | ||
| python map_solutions_connectors_tables.py | ||
|
|
||
| # Note: Documentation generation removed - docs are now hosted in a separate repo | ||
| # See: https://github.com/oshezaf/sentinelninja/tree/main/Solutions%20Docs | ||
|
|
||
| - name: Check for changes | ||
| id: check_changes | ||
| run: | | ||
| if git diff --quiet "Tools/Solutions Analyzer/solutions_connectors_tables_mapping.csv" "Tools/Solutions Analyzer/solutions_connectors_tables_issues_and_exceptions_report.csv"; then | ||
| echo "changed=false" >> $GITHUB_OUTPUT | ||
| else | ||
| echo "changed=true" >> $GITHUB_OUTPUT | ||
| fi | ||
|
|
||
| - name: Create Pull Request | ||
| if: steps.check_changes.outputs.changed == 'true' | ||
| id: create_pr | ||
| uses: peter-evans/create-pull-request@v6 | ||
Check warningCode scanning / CodeQL Unpinned tag for a non-immutable Action in workflow Medium
Unpinned 3rd party Action 'Update Solutions Analyzer CSV Files' step
Uses Step: create_pr Error loading related location Loading |
||
| with: | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
| commit-message: 'chore: Update Solutions Analyzer CSV files' | ||
| branch: solutions-analyzer-update | ||
| delete-branch: true | ||
| title: 'chore: Update Solutions Analyzer CSV files' | ||
| body: | | ||
| ## Automated Solutions Analyzer Update | ||
|
|
||
| This PR contains automated updates to: | ||
| - Solutions connector-to-tables mapping CSV | ||
| - Solutions issues and exceptions report CSV | ||
|
|
||
| Generated by the Solutions Analyzer workflow. | ||
|
|
||
| **Note:** Documentation is now hosted separately at https://github.com/oshezaf/sentinelninja | ||
|
|
||
| **Triggered by:** ${{ github.event_name }} | ||
| **Workflow run:** ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }} | ||
| labels: automated | ||
| add-paths: | | ||
| Tools/Solutions Analyzer/solutions_connectors_tables_mapping.csv | ||
| Tools/Solutions Analyzer/solutions_connectors_tables_issues_and_exceptions_report.csv | ||
|
|
||
| - name: Enable auto-merge | ||
| if: steps.check_changes.outputs.changed == 'true' && steps.create_pr.outputs.pull-request-number != '' | ||
| run: | | ||
| gh pr merge ${{ steps.create_pr.outputs.pull-request-number }} --auto --squash | ||
| env: | ||
| GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| - name: Create summary | ||
| if: steps.check_changes.outputs.changed == 'true' | ||
| run: | | ||
| echo "### Solutions Analyzer Pull Request Created :white_check_mark:" >> $GITHUB_STEP_SUMMARY | ||
| echo "" >> $GITHUB_STEP_SUMMARY | ||
| echo "A pull request has been created with updated CSV files." >> $GITHUB_STEP_SUMMARY | ||
| echo "" >> $GITHUB_STEP_SUMMARY | ||
| echo "**Modified files:**" >> $GITHUB_STEP_SUMMARY | ||
| echo "- Tools/Solutions Analyzer/solutions_connectors_tables_mapping.csv" >> $GITHUB_STEP_SUMMARY | ||
| echo "- Tools/Solutions Analyzer/solutions_connectors_tables_issues_and_exceptions_report.csv" >> $GITHUB_STEP_SUMMARY | ||
| echo "" >> $GITHUB_STEP_SUMMARY | ||
| echo "**Note:** Documentation is hosted at https://github.com/oshezaf/sentinelninja" >> $GITHUB_STEP_SUMMARY | ||
|
|
||
| - name: No changes summary | ||
| if: steps.check_changes.outputs.changed == 'false' | ||
| run: | | ||
| echo "### Solutions Analyzer :information_source:" >> $GITHUB_STEP_SUMMARY | ||
| echo "" >> $GITHUB_STEP_SUMMARY | ||
| echo "No changes detected. CSV files are already up-to-date." >> $GITHUB_STEP_SUMMARY | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Check warning
Code scanning / CodeQL
Code injection Medium
Copilot Autofix
AI 3 months ago
General approach: Avoid using
${{ ... }}expressions that embed user-controlled values directly insiderun:shell scripts. Instead, assign the untrusted value to an environment variable using workflow expression syntax, and then reference it using native shell variable expansion ($VAR) within the script.Concrete fix here: For the “Commit updated bundles” step, move
${{ github.event.pull_request.head.ref }}into a new env variable (e.g.,PR_HEAD_REF) and then change thegit pushline to use$PR_HEAD_REF. This prevents GitHub from interpolating the untrusted value into the shell script; the shell now receives the value as normal data via the environment, which is the recommended pattern. Functionality remains identical becausegit push origin <branch>is the same call; we’re only changing how the branch name gets passed in.Changes needed in
.github/workflows/aws-s3-bundle-update.yaml:In the “Commit updated bundles” step (lines 77–106), add an environment variable, e.g.:
Update line 101 from:
git push origin ${{ github.event.pull_request.head.ref }}to:
git push origin "$PR_HEAD_REF"No additional imports or external dependencies are needed.