Skip to content

[Backport 2.4] fixed GetFindings not searching all indices; fixed proper deletion of…#136

Merged
eirsep merged 1 commit into2.4from
backport/backport-122-to-2.4
Nov 9, 2022
Merged

[Backport 2.4] fixed GetFindings not searching all indices; fixed proper deletion of…#136
eirsep merged 1 commit into2.4from
backport/backport-122-to-2.4

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport 8dd1c9f from #122

#122)

* fixed GetFindings not searching all indices; fixed proper deletion of old history indices

Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
(cherry picked from commit 8dd1c9f)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team November 9, 2022 20:17
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 9, 2022

Codecov Report

Merging #136 (8dd1c9f) into 2.4 (5de2a8d) will increase coverage by 0.05%.
The diff coverage is 13.23%.

❗ Current head 8dd1c9f differs from pull request most recent head e44ccb2. Consider uploading reports for the commit e44ccb2 to get more accurate results

@@             Coverage Diff              @@
##                2.4     #136      +/-   ##
============================================
+ Coverage     39.24%   39.29%   +0.05%     
- Complexity      882      891       +9     
============================================
  Files           176      176              
  Lines          6659     6683      +24     
  Branches        809      811       +2     
============================================
+ Hits           2613     2626      +13     
- Misses         3807     3816       +9     
- Partials        239      241       +2     
Impacted Files Coverage Δ
...rch/securityanalytics/SecurityAnalyticsPlugin.java 0.00% <0.00%> (ø)
...arch/securityanalytics/action/AckAlertsAction.java 0.00% <ø> (ø)
...ityanalytics/action/CreateIndexMappingsAction.java 0.00% <ø> (ø)
...securityanalytics/action/DeleteDetectorAction.java 0.00% <ø> (ø)
...rch/securityanalytics/action/DeleteRuleAction.java 0.00% <ø> (ø)
...arch/securityanalytics/action/GetAlertsAction.java 0.00% <ø> (ø)
...ch/securityanalytics/action/GetDetectorAction.java 100.00% <ø> (ø)
...ch/securityanalytics/action/GetFindingsAction.java 0.00% <ø> (ø)
...curityanalytics/action/GetIndexMappingsAction.java 0.00% <ø> (ø)
...ecurityanalytics/action/GetMappingsViewAction.java 0.00% <ø> (ø)
... and 59 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@eirsep eirsep merged commit 2a27e0f into 2.4 Nov 9, 2022
@github-actions github-actions bot deleted the backport/backport-122-to-2.4 branch November 9, 2022 20:51
riysaxen-amzn pushed a commit to riysaxen-amzn/security-analytics that referenced this pull request Feb 20, 2024
…present (opensearch-project#136)

* disable notification option if no notif plugin

Signed-off-by: Aleksandar Djindjic <djindjic@gmail.com>

* use fragment and &nbsp;

Signed-off-by: Aleksandar Djindjic <djindjic@gmail.com>

* reference github issue in TODO

Signed-off-by: Aleksandar Djindjic <djindjic@gmail.com>

* change console.error to teaster notifications

Signed-off-by: Aleksandar Djindjic <djindjic@gmail.com>

* add learn more link Resolves opensearch-project#137

Signed-off-by: Aleksandar Djindjic <djindjic@gmail.com>

* fix console.error log

Signed-off-by: Aleksandar Djindjic <djindjic@gmail.com>

* Adjusted OSD version used by test workflows. (opensearch-project#149)

* Adjusted OSD version used by test workflows.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Fixed env variable call.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* Revised comment to include github issue link.

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
Signed-off-by: Aleksandar Djindjic <djindjic@gmail.com>

* use console.warn(s)

Signed-off-by: Aleksandar Djindjic <djindjic@gmail.com>

* extract response logic to helper method

Signed-off-by: Aleksandar Djindjic <djindjic@gmail.com>

Signed-off-by: Aleksandar Djindjic <djindjic@gmail.com>
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
Co-authored-by: AWSHurneyt <hurneyt@amazon.com>
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.

3 participants