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

Fixing bug in RequestUtil.ProcessRequest #58

Merged
merged 3 commits into from
Feb 27, 2019
Merged

Commits on Feb 26, 2019

  1. Fixing bug in RequestUtil.ProcessRequest

    Don't access the trace context in `RequestUtil.ProcessRequest` when tracing is disabled. See aws#57 for more information.
    djluck committed Feb 26, 2019
    Configuration menu
    Copy the full SHA
    d952948 View commit details
    Browse the repository at this point in the history
  2. Added unit tests to verify HTTP requests can be sent when tracing is …

    …disabled. In the process, added disposal of HttpWebResponse/ HttpResponseMessage objects to avoid connection timeouts when sending multiple HTTP requests to the same endpoint.
    James Luck committed Feb 26, 2019
    Configuration menu
    Copy the full SHA
    de9193f View commit details
    Browse the repository at this point in the history

Commits on Feb 27, 2019

  1. Adding a test to verify GetAsyncResponseTraced doesn't fail when XRay…

    … tracing is disabled + fixing up naming of tests to be consistent with existing naming style.
    James Luck committed Feb 27, 2019
    Configuration menu
    Copy the full SHA
    2d9197c View commit details
    Browse the repository at this point in the history