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

Fix wrong request arguments handling for httpc adapter #38

Merged
merged 1 commit into from
Jan 13, 2016

Conversation

zoldar
Copy link
Contributor

@zoldar zoldar commented Jan 12, 2016

The body was extracted from the wrong place in generate_keys_for_request. By the way, I've cleaned the function up a bit.

@zoldar zoldar force-pushed the wrong-httpc-request-handling branch from 55a63af to 99bb044 Compare January 12, 2016 13:13
@aerosol
Copy link

aerosol commented Jan 13, 2016

+1, the checks failed because of coverage posting to 3rd party failed

parroty added a commit that referenced this pull request Jan 13, 2016
Fix wrong request arguments handling for httpc adapter
@parroty parroty merged commit 8355f1d into parroty:master Jan 13, 2016
@parroty
Copy link
Owner

parroty commented Jan 13, 2016

Thanks, and sorry about the flaky build.

@parroty
Copy link
Owner

parroty commented Jan 13, 2016

Travis seems slow and taking time to complete testing.. I'll be pushing hex package once after completed.

@aerosol
Copy link

aerosol commented Jan 13, 2016

Thanks!

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

3 participants