Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Correctly log expected errors when fetching server keys #4847

Merged
merged 2 commits into from
Mar 11, 2019

Conversation

erikjohnston
Copy link
Member

We now wrap these errors in KeyLookupError

@erikjohnston erikjohnston marked this pull request as ready for review March 11, 2019 14:14
@erikjohnston erikjohnston requested a review from a team March 11, 2019 14:14
@codecov
Copy link

codecov bot commented Mar 11, 2019

Codecov Report

Merging #4847 into develop will increase coverage by 0.04%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop    #4847      +/-   ##
===========================================
+ Coverage    75.31%   75.35%   +0.04%     
===========================================
  Files          340      340              
  Lines        34924    34924              
  Branches      5718     5718              
===========================================
+ Hits         26302    26318      +16     
+ Misses        7009     6993      -16     
  Partials      1613     1613

Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

plausible

@erikjohnston erikjohnston merged commit 9c0e6e8 into develop Mar 11, 2019
@erikjohnston erikjohnston deleted the erikj/fix_stacktrace_keyring branch January 9, 2020 15:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants