Skip to content

TacitRed-IOC-CrowdStrike v3.0.1: Add missing playbook discovery tags (hidden-SentinelTemplateName/Version)#13710

Closed
mazamizo21 wants to merge 2 commits intoAzure:masterfrom
Data443:feature/tacitred-cs-hidden-tags
Closed

TacitRed-IOC-CrowdStrike v3.0.1: Add missing playbook discovery tags (hidden-SentinelTemplateName/Version)#13710
mazamizo21 wants to merge 2 commits intoAzure:masterfrom
Data443:feature/tacitred-cs-hidden-tags

Conversation

@mazamizo21
Copy link
Contributor

Problem

After installing TacitRed-IOC-CrowdStrike from Content Hub, the playbook template does not appear in Automation > Playbook templates. The solution shows as "Installed" but the Configuration page is empty — no wizard, no resource group selection.

This is the same issue that affected TacitRed-IOC-SentinelOne (already fixed and live in Content Hub).

Root Cause

The v3.0.0 mainTemplate.json in the Package is missing hidden-SentinelTemplateName and hidden-SentinelTemplateVersion tags on the Logic App resource. These tags are how Sentinel discovers playbook templates — without them, the playbook is invisible.

Note: PR #13641 (v3.0.1) was previously merged but only included the Solution JSON file. The Package/mainTemplate.json and zip were not part of that merge. This PR adds them.

Changes (3.0.0 → 3.0.1)

Fix Before (3.0.0) After (3.0.1)
hidden-SentinelTemplateName MISSING TacitRedToCrowdStrike
hidden-SentinelTemplateVersion MISSING 1.0
location parameter Hardcoded resourceGroup().location workspace-location-inline (matches 489/492 solutions)
TacitRed_Domain param Present (unused) Removed
solutionId data443.azure-sentinel-... data443riskmitigationinc1761580347231.azure-sentinel-... (matches Partner Center offer)
API versions 2025-09-01 (future-dated) 2023-04-01-preview
Publisher name TacitRed Data443 Risk Mitigation, Inc.

Files Modified

  • Solutions/TacitRed-IOC-CrowdStrike/Package/mainTemplate.json — all fixes above
  • Solutions/TacitRed-IOC-CrowdStrike/Package/3.0.1.zip — rebuilt package
  • Solutions/TacitRed-IOC-CrowdStrike/ReleaseNotes.md — added v3.0.1 entry

Old 3.0.0.zip preserved as required.

@mazamizo21 mazamizo21 requested review from a team as code owners February 26, 2026 10:10
@v-maheshbh v-maheshbh added the Solution Solution specialty review needed label Feb 26, 2026
@v-shukore
Copy link
Contributor

Hi @mazamizo21, please resolve branch conflicts. Thanks!

…and deployment fixes

Changes from 3.0.0 to 3.0.1:
- Added hidden-SentinelTemplateName and hidden-SentinelTemplateVersion tags
  (without these, playbook template does not appear in Automation > Playbook templates)
- Fixed location parameter: removed hardcoded location, use workspace-location-inline
- Removed unused TacitRed_Domain parameter and URI filter
- Fixed solutionId to match Partner Center offer ID
- Updated API versions from future-dated 2025-09-01 to 2023-04-01-preview
- Updated publisher name to Data443 Risk Mitigation, Inc.

Note: PR Azure#13641 (v3.0.1) was previously merged but only included the Solution JSON —
the Package/mainTemplate.json and zip were not part of that merge. This PR adds them.
@mazamizo21 mazamizo21 force-pushed the feature/tacitred-cs-hidden-tags branch from 1a4dcb0 to 02c9b36 Compare March 2, 2026 09:01
@mazamizo21 mazamizo21 requested review from a team as code owners March 2, 2026 09:01
- name: Create Pull Request
if: steps.check_changes.outputs.changed == 'true'
id: create_pr
uses: peter-evans/create-pull-request@v6

Check warning

Code 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
uses 'peter-evans/create-pull-request' with ref 'v6', not a pinned commit hash
@mazamizo21
Copy link
Contributor Author

Closing this PR — the fix is already on master.

After reviewing, I confirmed that the Feb 17 merge (commit 3193698) did include the updated mainTemplate.json and 3.0.1.zip with all the required hidden-SentinelTemplateName/hidden-SentinelTemplateVersion tags. Both the loose template and the zip have the fix.

Apologies for the noisy diff — the branch picked up unrelated local files during the rebase. No changes needed.

@mazamizo21 mazamizo21 closed this Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Solution Solution specialty review needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants