Skip to content

Conversation

@cviecco
Copy link
Contributor

@cviecco cviecco commented Oct 28, 2019

This request is about preparing the eventmond code to support TLS. To do so this patch does 2 things:

  1. Splits the adding of the eventmond paths to the default mux to separate public function in the API. (and also uses this split)
  2. Explicitly creates a new server with the default server mux to enable reasonable timeouts on the connections. (these same timeouts will be used for the TLS version).

@cviecco cviecco requested a review from rgooch October 28, 2019 22:36
Copy link
Contributor

@rgooch rgooch left a comment

Choose a reason for hiding this comment

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

Can you please add a description of the change including the motivation?

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.

2 participants