Skip to content

Conversation

@sujithvm
Copy link
Contributor

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@sujithvm sujithvm requested a review from a team as a code owner May 14, 2020 23:48
@sujithvm sujithvm changed the title Fix routes and add nodesdn to validation api Fix routes in master May 15, 2020
@codecov
Copy link

codecov bot commented May 15, 2020

Codecov Report

Merging #461 into master will decrease coverage by 0.10%.
The diff coverage is 96.20%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #461      +/-   ##
============================================
- Coverage     62.34%   62.23%   -0.11%     
- Complexity     2902     2927      +25     
============================================
  Files           226      225       -1     
  Lines         16185    16142      -43     
  Branches       3017     3015       -2     
============================================
- Hits          10090    10046      -44     
+ Misses         4526     4525       -1     
- Partials       1569     1571       +2     
Impacted Files Coverage Δ Complexity Δ
...arch/security/dlic/rest/api/AbstractApiAction.java 73.89% <ø> (-0.12%) 57.00 <0.00> (ø)
...csearch/security/configuration/DlsQueryParser.java 78.00% <50.00%> (ø) 8.00 <0.00> (ø)
...rity/ssl/rest/OpenDistroSecuritySSLInfoAction.java 72.91% <75.00%> (+0.57%) 4.00 <2.00> (+2.00)
...earch/security/auditlog/impl/AbstractAuditLog.java 70.36% <83.33%> (-1.04%) 88.00 <0.00> (-2.00)
...asticsearch/security/OpenDistroSecurityPlugin.java 82.27% <100.00%> (-0.04%) 92.00 <1.00> (ø)
...ty/configuration/ConfigurationLoaderSecurity7.java 65.74% <100.00%> (ø) 9.00 <0.00> (ø)
...earch/security/dlic/rest/api/AccountApiAction.java 83.33% <100.00%> (-0.23%) 14.00 <2.00> (+1.00) ⬇️
.../security/dlic/rest/api/ActionGroupsApiAction.java 100.00% <100.00%> (ø) 8.00 <2.00> (+1.00)
...curity/dlic/rest/api/AuthTokenProcessorAction.java 57.14% <100.00%> (ø) 3.00 <2.00> (+1.00)
...ch/security/dlic/rest/api/FlushCacheApiAction.java 63.33% <100.00%> (-3.34%) 10.00 <2.00> (+1.00) ⬇️
... and 23 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ed6f901...0c4446b. Read the comment docs.

@sujithvm sujithvm requested review from hardik-k-shah and vrozov May 15, 2020 22:13
@sujithvm sujithvm merged commit dcca8ec into opensearch-project:master May 16, 2020
lbreinig pushed a commit to lbreinig/security that referenced this pull request Dec 23, 2021
- Fix registering routes for Rest handlers for ES 7.7
wuychn pushed a commit to ochprince/security that referenced this pull request Mar 16, 2023
- Fix registering routes for Rest handlers for ES 7.7
gaobinlong pushed a commit to gaobinlong/security that referenced this pull request Aug 30, 2023
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