Closed WindzCUHK closed 4 years ago
Merging #42 into master will increase coverage by
0.25%
. The diff coverage is97.74%
.
@@ Coverage Diff @@
## master #42 +/- ##
==========================================
+ Coverage 90.60% 90.86% +0.25%
==========================================
Files 13 13
Lines 1096 1127 +31
==========================================
+ Hits 993 1024 +31
Misses 79 79
Partials 24 24
Impacted Files | Coverage Δ | |
---|---|---|
config/config.go | 88.23% <ø> (ø) |
|
usecase/tenantd.go | 83.54% <89.47%> (+1.59%) |
:arrow_up: |
service/server.go | 95.31% <91.66%> (ø) |
|
handler/handlers.go | 100.00% <100.00%> (ø) |
|
main.go | 53.84% <100.00%> (+17.48%) |
:arrow_up: |
router/routes.go | 100.00% <100.00%> (ø) |
|
service/access.go | 93.92% <100.00%> (ø) |
|
service/role.go | 94.41% <100.00%> (ø) |
|
service/svccert.go | 92.85% <100.00%> (ø) |
|
service/tls.go | 94.73% <100.00%> (-0.14%) |
:arrow_down: |
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 2123f92...65012b3. Read the comment docs.
Description
changes
config.go
v2.0.0
/test/data
SA1019: t.BuildNameToCertificate is deprecated: NameToCertificate only allows associating a single certificate with a given name. Leave that field nil to let the library select the first compatible chain from Certificates. (staticcheck)
/svccert
/access-token
=>/accesstoken
roletoken
=>role token
https://github.com/yahoojapan/athenz-client-sidecar/blob/34243ebf47a378697799c1acf6071cf4f098f267/docs/example_config.yaml#L1-L69
Type of change
Flags
Related issue/PR
Checklist
[major]
/[minor]
/[patch]
/[skip]
in the PR titleChecklist for maintainer
Squash and merge
[major]
/[minor]
/[patch]
/[skip]