Avoid logging pings. A reportRequest function is passed into HTTP1OperationRequestHandler.handle allowing the conforming type to choose when to report the request start to the OperationTraceContext. By default (in StandardHTTP1OperationRequestHandler) this function will only be called if the request is not a ping.
StandardHTTP1ResponseHandler. handleComplete takes an additional reportCompletion parameter, which if false will not report request completion to the OperationTraceContext.
Reduce a number of logging statements from debug to 'trace`
Reduce the log statement in StandardHTTP1ResponseHandler.complete from info to trace as it is duplicated by any logging in the OperationTraceContext
This is a breaking change and will be part of the next major version, 3.X.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Issue #, if available:
Description of changes:
reportRequestfunction is passed intoHTTP1OperationRequestHandler.handleallowing the conforming type to choose when to report the request start to theOperationTraceContext. By default (inStandardHTTP1OperationRequestHandler) this function will only be called if the request is not a ping.StandardHTTP1ResponseHandler. handleCompletetakes an additionalreportCompletionparameter, which if false will not report request completion to theOperationTraceContext.debugto 'trace`StandardHTTP1ResponseHandler.completefrominfototraceas it is duplicated by any logging in theOperationTraceContextThis is a breaking change and will be part of the next major version, 3.X.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.