Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add .name to outbound tls for logs #2492

Merged
merged 1 commit into from Aug 10, 2018
Merged

Add .name to outbound tls for logs #2492

merged 1 commit into from Aug 10, 2018

Conversation

analogic
Copy link
Collaborator

Fixes #2491

Changes proposed in this pull request:

  • add .name to outbound tls

Checklist:

  • docs updated
  • tests updated
  • Changes updated

@codecov
Copy link

codecov bot commented Aug 10, 2018

Codecov Report

Merging #2492 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2492      +/-   ##
==========================================
+ Coverage   59.34%   59.35%   +<.01%     
==========================================
  Files          28       28              
  Lines        6194     6195       +1     
  Branches     1540     1540              
==========================================
+ Hits         3676     3677       +1     
  Misses       2518     2518
Impacted Files Coverage Δ
outbound/tls.js 56.66% <100%> (+0.73%) ⬆️

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 3a175f2...9fabd13. Read the comment docs.

@msimerson msimerson merged commit 275eeee into haraka:master Aug 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants