Skip to content
This repository has been archived by the owner on Jun 14, 2023. It is now read-only.

Support TLS with gRPCReporter #62

Merged
merged 8 commits into from
Jun 15, 2020
Merged

Support TLS with gRPCReporter #62

merged 8 commits into from
Jun 15, 2020

Conversation

arugal
Copy link
Member

@arugal arugal commented Jun 15, 2020

No description provided.

@arugal arugal added the enhancement New feature or request label Jun 15, 2020
@arugal arugal added this to the 0.4.0 milestone Jun 15, 2020
@arugal arugal requested a review from wu-sheng June 15, 2020 14:43
@arugal arugal changed the title Support TLS for gRPCReporter Support TLS with gRPCReporter Jun 15, 2020
@codecov-commenter
Copy link

codecov-commenter commented Jun 15, 2020

Codecov Report

Merging #62 into master will decrease coverage by 0.38%.
The diff coverage is 40.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
- Coverage   70.17%   69.79%   -0.39%     
==========================================
  Files          14       14              
  Lines         674      682       +8     
==========================================
+ Hits          473      476       +3     
- Misses        160      165       +5     
  Partials       41       41              
Impacted Files Coverage Δ
reporter/grpc.go 61.34% <40.00%> (-1.03%) ⬇️

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 7ba623d...c8bfb8b. Read the comment docs.

@arugal arugal merged commit e61c0b1 into SkyAPM:master Jun 15, 2020
@arugal arugal deleted the tls branch June 15, 2020 15:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants