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

Add referrerpolicy #28

Merged
merged 2 commits into from
Sep 7, 2022
Merged

Add referrerpolicy #28

merged 2 commits into from
Sep 7, 2022

Conversation

khems
Copy link
Contributor

@khems khems commented Sep 6, 2022

Adds referrerpolicy.

I couldn't find a contributing section on this repository.

We'll need to bump the version number.

@lavrton
Copy link
Member

lavrton commented Sep 6, 2022

Interesting. What is your use case for that property? I never used it before. Do you have a case where you need to manually change it?

@khems
Copy link
Contributor Author

khems commented Sep 7, 2022

Yes, I needed to change it so it worked with the policy set on our S3 bucket where we are storing the image.

I'm surprised nobody else has run into this issue, and I'm sure others would benefit from the flexibility.

@lavrton lavrton merged commit 4f5ebba into konvajs:master Sep 7, 2022
@lavrton
Copy link
Member

lavrton commented Sep 7, 2022

Published [email protected]

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.

2 participants