Skip to content
This repository has been archived by the owner on Jul 10, 2022. It is now read-only.

fix Content-Type to same type as official repository #146

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mtanda
Copy link

@mtanda mtanda commented Jul 17, 2018

I think the Content-Type should be same as official repository.
https://gist.github.com/mtanda/5d02c0b2b6a8f8abfd2bfe1ec519641d

The motivation to fix Content-Type is, I want to access S3 hosted repository from API Gateway.
https://docs.aws.amazon.com/apigateway/latest/developerguide/integrating-api-with-aws-services-s3.html

I'm not sure why charset=binary is set, but it cause error when I access S3 via API Gateway.
The error message is Charset is invalid or illegal.
When I remove charset=binary from Content-Type, I can access S3 repository.

@mtanda
Copy link
Author

mtanda commented Jul 18, 2018

@krobertson I'm terribly sorry to bother you, would you check this?

@brandonryan
Copy link

I am having this issue too. Cant seem to get around it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants