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

Support HTTP headers #79

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Support HTTP headers #79

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jul 13, 2018

Adds support for HTTP headers when fetching an image via the headers prop, as you would be able to do with the headers option for the source prop for RN Image.

@wcandillon
Copy link
Owner

@Versalife Thank you so much for contributing this. Should the http headers by passed via the header property of source? This would mean that now the source would be source not URI, which would make it a bit more symetric with <Image> and then it would support this change, do you agree?

@ghost
Copy link
Author

ghost commented Jul 29, 2018

That would be better yeah, I agree. I'll have a go at making that change

@wcandillon
Copy link
Owner

wcandillon commented Jul 29, 2018 via email

@ghost
Copy link
Author

ghost commented Jul 29, 2018

That's done - I've just changed the interface of the component and left the CacheManager to deal with the uri and headers only to keep things simple.

@AlessandroAries
Copy link

@wcandillon can you resolve the conflict and merge this pr, I would like to see this change come through :)

@wcandillon
Copy link
Owner

@Versalife Sorry for taking so much time on this. Could you rebase your PR with the latest version of the module?

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.

3 participants