Skip to content

[Backport 2.x] findingsDto assign detectorId bug#200

Merged
sbcd90 merged 1 commit into2.xfrom
backport/backport-102-to-2.x
Dec 15, 2022
Merged

[Backport 2.x] findingsDto assign detectorId bug#200
sbcd90 merged 1 commit into2.xfrom
backport/backport-102-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport 1ef30c8 from #102

Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>

Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
(cherry picked from commit 1ef30c8)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team December 15, 2022 01:58
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #200 (ab97442) into 2.x (eda3cef) will increase coverage by 0.04%.
The diff coverage is 80.00%.

@@             Coverage Diff              @@
##                2.x     #200      +/-   ##
============================================
+ Coverage     39.00%   39.05%   +0.04%     
- Complexity      884      890       +6     
============================================
  Files           176      176              
  Lines          6717     6722       +5     
  Branches        820      820              
============================================
+ Hits           2620     2625       +5     
+ Misses         3856     3855       -1     
- Partials        241      242       +1     
Impacted Files Coverage Δ
...lytics/transport/TransportIndexDetectorAction.java 0.00% <0.00%> (ø)
...opensearch/securityanalytics/util/RuleIndices.java 0.00% <0.00%> (ø)
...pensearch/securityanalytics/action/FindingDto.java 37.20% <100.00%> (+12.20%) ⬆️
...g/opensearch/securityanalytics/model/Detector.java 69.89% <0.00%> (-0.70%) ⬇️

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

@sbcd90 sbcd90 merged commit e91f2a2 into 2.x Dec 15, 2022
@github-actions github-actions bot deleted the backport/backport-102-to-2.x branch December 15, 2022 02:15
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