Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

merge to master #289

Merged
merged 15 commits into from
Oct 28, 2020
Merged

merge to master #289

merged 15 commits into from
Oct 28, 2020

Conversation

skkosuri-amzn
Copy link
Contributor

Issue #, if available:

Resolve user, backendroles, roles from threadcontext instead of /authinfo
Description of changes:

Resolve user, backendroles, roles from threadcontext instead of /authinfo

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

skkosuri-amzn and others added 15 commits September 9, 2020 15:49
…tination response, 3/ handle null in GetMonitorRequest
…info rest call (#288)

* Resolve user, backendroles, roles from threadcontext instead of /authinfo rest call
* remove unwanted entry
* move to 1.11.0.1 commons and ktlint
@codecov
Copy link

codecov bot commented Oct 28, 2020

Codecov Report

Merging #289 into master will increase coverage by 0.78%.
The diff coverage is 41.02%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #289      +/-   ##
============================================
+ Coverage     79.42%   80.21%   +0.78%     
+ Complexity      197      196       -1     
============================================
  Files           151      151              
  Lines          5211     5146      -65     
  Branches        673      668       -5     
============================================
- Hits           4139     4128      -11     
+ Misses          717      662      -55     
- Partials        355      356       +1     
Impacted Files Coverage Δ Complexity Δ
...csearch/alerting/action/IndexDestinationRequest.kt 100.00% <ø> (ø) 0.00 <0.00> (ø)
...asticsearch/alerting/action/IndexMonitorRequest.kt 100.00% <ø> (ø) 0.00 <0.00> (ø)
.../alerting/transport/TransportIndexMonitorAction.kt 59.09% <25.00%> (+4.19%) 0.00 <0.00> (ø)
...rch/alerting/transport/TransportGetAlertsAction.kt 75.38% <36.36%> (+13.74%) 0.00 <0.00> (ø)
...erting/transport/TransportGetDestinationsAction.kt 76.05% <36.36%> (+12.76%) 0.00 <0.00> (ø)
...rting/transport/TransportIndexDestinationAction.kt 54.95% <36.36%> (+3.62%) 0.00 <0.00> (ø)
...alerting/transport/TransportSearchMonitorAction.kt 58.33% <42.85%> (+29.76%) 0.00 <0.00> (ø)
...ndistroforelasticsearch/alerting/AlertingPlugin.kt 91.66% <100.00%> (+0.85%) 0.00 <0.00> (ø)
...relasticsearch/alerting/action/GetAlertsRequest.kt 100.00% <100.00%> (ø) 0.00 <0.00> (ø)
...icsearch/alerting/action/GetDestinationsRequest.kt 100.00% <100.00%> (ø) 0.00 <0.00> (ø)
... and 11 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 9986272...4c67ae2. Read the comment docs.

@skkosuri-amzn skkosuri-amzn merged commit fc974fd into master Oct 28, 2020
@bowenlan-amzn bowenlan-amzn added enhancement New feature or request bug Something isn't working and removed enhancement New feature or request labels Nov 17, 2020
@qreshi qreshi deleted the aes-master branch December 19, 2020 00:45
tlfeng pushed a commit that referenced this pull request Feb 6, 2021
* Resolve user, backendroles, roles from threadcontext instead of /authinfo rest call (#288)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants