Skip to content

Fix pp apollo#75

Merged
roarinpenguin merged 2 commits intonatesmalley:mainfrom
jmorascalyr:fix-pp-apollo
Mar 3, 2026
Merged

Fix pp apollo#75
roarinpenguin merged 2 commits intonatesmalley:mainfrom
jmorascalyr:fix-pp-apollo

Conversation

@jmorascalyr
Copy link
Collaborator

No description provided.

…update classification ID

- Changed email alert resource UID from shared GUID to fresh UUID (uuid.uuid4()) to prevent S1 UAM from silently dropping site-scoped alerts with static resource UIDs
- Updated proofpoint_email_alert.json s1_classification_id from 1 to 28
- Removed email_asset_uid caching logic that was causing alert deduplication issues
…rrelation scenario execution

- Added suppress_alerts and strip_helios_prefix boolean flags to CorrelationRunRequest model
- Updated start_correlation_scenario() and _execute_correlation_scenario() to accept and pass new alert control flags
- Modified send_phase_alert() to conditionally strip "HELIOS - " prefix from alert titles when strip_helios_prefix=True
- Added environment variable fallbacks (SCENARIO_SUPPRESS_ALERTS, SCENARIO_STRIP_HEL
@roarinpenguin roarinpenguin merged commit a6f9bb6 into natesmalley:main Mar 3, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants