Skip to content
This repository was archived by the owner on Oct 11, 2024. It is now read-only.

Comments

Don't double encrypt#1439

Merged
gdbelvin merged 2 commits intogoogle:masterfrom
gdbelvin:mon_creds
Jan 20, 2020
Merged

Don't double encrypt#1439
gdbelvin merged 2 commits intogoogle:masterfrom
gdbelvin:mon_creds

Conversation

@gdbelvin
Copy link
Contributor

Fixup from #1435

@gdbelvin gdbelvin requested a review from a team as a code owner January 20, 2020 15:07
@gdbelvin gdbelvin requested a review from mhutchinson January 20, 2020 15:07
@codecov
Copy link

codecov bot commented Jan 20, 2020

Codecov Report

Merging #1439 into master will increase coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1439      +/-   ##
==========================================
+ Coverage   66.44%   66.54%   +0.09%     
==========================================
  Files          54       54              
  Lines        4026     4026              
==========================================
+ Hits         2675     2679       +4     
+ Misses        959      957       -2     
+ Partials      392      390       -2
Impacted Files Coverage Δ
core/sequencer/server.go 73.61% <0%> (+0.65%) ⬆️
core/sequencer/trillian_client.go 61.42% <0%> (+2.85%) ⬆️

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 e6f4dd0...53064a0. Read the comment docs.

@gdbelvin gdbelvin merged commit ac3b6cf into google:master Jan 20, 2020
@gdbelvin gdbelvin deleted the mon_creds branch January 20, 2020 15:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants