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

cargohold: pin newer amazonka with Content-Length patch #1217

Merged
merged 1 commit into from
Oct 8, 2020

Conversation

mheinzel
Copy link
Contributor

@mheinzel mheinzel commented Oct 7, 2020

See https://github.com/zinfra/backend-issues/issues/1659

This pins 5d31aa49177e33ff1d6a00255f28e4ef3b711f7d, which is now the most recent commit on wireapp/amazonka develop. If we want to merge this, I can also make a new release in our amazonka fork to make it clearer which version we're using (like this one).

Copy link
Member

@jschaul jschaul left a comment

Choose a reason for hiding this comment

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

This works nicely.

@mheinzel
Copy link
Contributor Author

mheinzel commented Oct 8, 2020

I made a release in our fork of amazonka, merging this for now.

@jschaul jschaul merged commit 4ce798e into develop Oct 8, 2020
@jschaul jschaul deleted the mheinzel/cargohold-s3-impl-compat branch October 8, 2020 16:43
jschaul added a commit that referenced this pull request Oct 13, 2020
@fisx fisx mentioned this pull request Oct 28, 2020
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

2 participants