Skip to content

ROX-26408: Encrypt Microsoft Sentinel Notifier Secret#12829

Merged
SimonBaeumer merged 6 commits intomasterfrom
sb/make-notifiers-encryption-aware
Oct 7, 2024
Merged

ROX-26408: Encrypt Microsoft Sentinel Notifier Secret#12829
SimonBaeumer merged 6 commits intomasterfrom
sb/make-notifiers-encryption-aware

Conversation

@SimonBaeumer
Copy link
Contributor

Description

ROX-26408: Encrypt Microsoft Sentinel Notifier Secret

change me!

User-facing documentation

  • CHANGELOG is updated OR update is not needed
  • documentation PR is created and is linked above OR is not needed

Automated testing

  • Added unit tests

@rhacs-bot
Copy link
Contributor

rhacs-bot commented Sep 27, 2024

Images are ready for the commit at bac04ad.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.6.x-650-gbac04ad5fc.

@codecov
Copy link

codecov bot commented Sep 27, 2024

Codecov Report

Attention: Patch coverage is 50.90909% with 27 lines in your changes missing coverage. Please review.

Project coverage is 48.20%. Comparing base (a0da1ae) to head (bac04ad).
Report is 16 commits behind head on master.

Files with missing lines Patch % Lines
central/notifiers/microsoftsentinel/sentinel.go 41.93% 15 Missing and 3 partials ⚠️
central/notifier/service/service_impl.go 0.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #12829      +/-   ##
==========================================
- Coverage   48.25%   48.20%   -0.06%     
==========================================
  Files        2441     2442       +1     
  Lines      175511   175702     +191     
==========================================
- Hits        84694    84693       -1     
- Misses      83997    84185     +188     
- Partials     6820     6824       +4     
Flag Coverage Δ
go-unit-tests 48.20% <50.90%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@charmik-redhat charmik-redhat left a comment

Choose a reason for hiding this comment

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

Might have to decrypt the credentials too when the notifier is used to send alert and logs.

Base automatically changed from sb/remove-unused-code to master September 30, 2024 13:20
@SimonBaeumer SimonBaeumer force-pushed the sb/make-notifiers-encryption-aware branch from 7051e81 to b5c5554 Compare October 4, 2024 09:07
@SimonBaeumer SimonBaeumer marked this pull request as ready for review October 4, 2024 09:09
@SimonBaeumer SimonBaeumer requested a review from a team as a code owner October 4, 2024 09:09
Copy link
Contributor

@charmik-redhat charmik-redhat 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.

@SimonBaeumer SimonBaeumer enabled auto-merge (squash) October 7, 2024 08:26
@SimonBaeumer SimonBaeumer merged commit fdd7a72 into master Oct 7, 2024
@SimonBaeumer SimonBaeumer deleted the sb/make-notifiers-encryption-aware branch October 7, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants