Make notifications optional 2.11 #770
Merged
Conversation
This file contains 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
* added threat intel feed support for detector creation Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated cypress workflow file Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated alerts; findings UX Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * refactored alert condition panel; update detector for intel feeds Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshots, mocks Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated workflow Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshot Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated UI; tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
amsiglan
changed the title
Codecov Report
@@ Coverage Diff @@
## 2.11 #770 +/- ##
==========================================
+ Coverage 31.17% 31.20% +0.03%
==========================================
Files 151 151
Lines 5008 5002 -6
Branches 910 908 -2
==========================================
Hits 1561 1561
+ Misses 3257 3252 -5
+ Partials 190 189 -1 |
amsiglan
added a commit
to amsiglan/security-analytics-dashboards-plugin
that referenced
this pull request
Oct 27, 2023
* Threat intel feed support for detector creation (opensearch-project#762) * added threat intel feed support for detector creation Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated cypress workflow file Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated alerts; findings UX Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * refactored alert condition panel; update detector for intel feeds Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshots, mocks Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated workflow Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshot Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated UI; tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed tests; make notification optional Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
70 tasks
amsiglan
added a commit
to amsiglan/security-analytics-dashboards-plugin
that referenced
this pull request
Nov 8, 2023
This reverts commit de6001a.
amsiglan
added a commit
to amsiglan/security-analytics-dashboards-plugin
that referenced
this pull request
Nov 8, 2023
This reverts commit de6001a. Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
AWSHurneyt
pushed a commit
that referenced
this pull request
Nov 8, 2023
* Revert "Make notifications optional 2.11 (#770)" This reverts commit de6001a. Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * Revert "Threat intel feed support for detector creation (#762) (#769)" This reverts commit 64982b9. Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * Revert "[Backport 2.11] Improved display of log types. (#766)" This reverts commit 25901f6. Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed cypress workflow Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
amsiglan
added a commit
to amsiglan/security-analytics-dashboards-plugin
that referenced
this pull request
Nov 22, 2023
* Threat intel feed support for detector creation (opensearch-project#762) * added threat intel feed support for detector creation Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated cypress workflow file Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated alerts; findings UX Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * refactored alert condition panel; update detector for intel feeds Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshots, mocks Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated workflow Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshot Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated UI; tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed tests; make notification optional Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
amsiglan
added a commit
that referenced
this pull request
Nov 29, 2023
* Make notifications optional 2.11 (#770) * Threat intel feed support for detector creation (#762) * added threat intel feed support for detector creation Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated cypress workflow file Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated alerts; findings UX Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * refactored alert condition panel; update detector for intel feeds Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshots, mocks Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated workflow Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshot Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated UI; tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed tests; make notification optional Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * revert notification change Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
amsiglan
added a commit
to amsiglan/security-analytics-dashboards-plugin
that referenced
this pull request
Feb 14, 2024
* Make notifications optional 2.11 (opensearch-project#770) * Threat intel feed support for detector creation (opensearch-project#762) * added threat intel feed support for detector creation Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated cypress workflow file Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated alerts; findings UX Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * refactored alert condition panel; update detector for intel feeds Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshots, mocks Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated workflow Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshot Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated UI; tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed tests; make notification optional Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * revert notification change Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
amsiglan
added a commit
that referenced
this pull request
Feb 14, 2024
* Make notifications optional (#796) * Make notifications optional 2.11 (#770) * Threat intel feed support for detector creation (#762) * added threat intel feed support for detector creation Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated cypress workflow file Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated alerts; findings UX Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * refactored alert condition panel; update detector for intel feeds Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshots, mocks Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated workflow Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshot Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated UI; tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed tests; make notification optional Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * revert notification change Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * supporting array of key/value under selection (#803) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * Correlation rules field matching and time window support (#804) * added ux for group by field and time window Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * improved validation Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * check for null values Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * addressed PR comments Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * validate fields only when group by enabled; update default window to 5 min (#810) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * do not send fields if empty (#817) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * Allow empty field name for supporting timeframe and keywords when validating rule during creation/update (#823) * allow empty field name for supporting timeframe and keywords Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed cypress test Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed update payload (#838) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * add already mapped log index fields to list of options (#845) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
amsiglan
added a commit
to amsiglan/security-analytics-dashboards-plugin
that referenced
this pull request
Feb 14, 2024
* Make notifications optional 2.11 (opensearch-project#770) * Threat intel feed support for detector creation (opensearch-project#762) * added threat intel feed support for detector creation Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated cypress workflow file Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated alerts; findings UX Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * refactored alert condition panel; update detector for intel feeds Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshots, mocks Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated workflow Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshot Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated UI; tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed tests; make notification optional Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * revert notification change Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
amsiglan
added a commit
that referenced
this pull request
Feb 14, 2024
* Make notifications optional (#796) * Make notifications optional 2.11 (#770) * Threat intel feed support for detector creation (#762) * added threat intel feed support for detector creation Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated cypress workflow file Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated alerts; findings UX Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * refactored alert condition panel; update detector for intel feeds Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshots, mocks Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated workflow Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated snapshot Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated UI; tests Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed tests; make notification optional Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * revert notification change Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * supporting array of key/value under selection (#803) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * Correlation rules field matching and time window support (#804) * added ux for group by field and time window Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * improved validation Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * check for null values Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * addressed PR comments Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * validate fields only when group by enabled; update default window to 5 min (#810) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * do not send fields if empty (#817) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * Allow empty field name for supporting timeframe and keywords when validating rule during creation/update (#823) * allow empty field name for supporting timeframe and keywords Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed cypress test Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * fixed update payload (#838) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Keep notification optional in create detector to block user from creating the detector. This is for the case when Notifications plugin is not installed in the frontend.
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.