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.

Sign up for GitHub

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

Jump to bottom

Fetch::Download: handle content decoding failure #159

Merged
merged 1 commit into from Dec 9, 2016
Merged

Fetch::Download: handle content decoding failure #159

merged 1 commit into from Dec 9, 2016

Conversation

stdweird
Copy link
Member

@stdweird stdweird commented Dec 8, 2016

Fixes #158

@stdweird stdweird added this to the 16.12 milestone Dec 8, 2016
ned21
ned21 approved these changes Dec 8, 2016
View reviewed changes
Copy link
Contributor

@ned21 ned21 left a comment

Choose a reason for hiding this comment

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

I don't understand the unit test error - looks unrelated? Code looks OK notwithstanding that.

@stdweird
Copy link
Member Author

stdweird commented Dec 9, 2016

retest this please

@stdweird
Copy link
Member Author

stdweird commented Dec 9, 2016

@ned21 must have been some glitch on jenkins. tests are ok now

@jrha jrha merged commit 346e861 into quattor:master Dec 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ned21 ned21 ned21 approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
16.12
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@stdweird @ned21 @jrha