Skip to content
This repository has been archived by the owner on Dec 18, 2018. It is now read-only.

Add a logging base class to Transport.Abstractions #2101

Closed
halter73 opened this issue Oct 6, 2017 · 1 comment
Closed

Add a logging base class to Transport.Abstractions #2101

halter73 opened this issue Oct 6, 2017 · 1 comment

Comments

@halter73
Copy link
Member

halter73 commented Oct 6, 2017

It could be beneficial to have methods in a transport logging base class for logging events like connection resets. This could serve as a minimum guideline for what should be logged by new transports and better enable cross-transport testing.

An alternative might be to move the more transport-agnostic logging like Connection(Error|Reset|Pause|Resume) into the Core project

@muratg
Copy link
Contributor

muratg commented Nov 10, 2017

Won't fix.

@muratg muratg closed this as completed Nov 10, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants