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

Allow Safari / WebKit to have SharedBuffer too #13

Merged
merged 1 commit into from
Jul 9, 2023

Conversation

WebReflection
Copy link
Contributor

This MR enables CORP implicitly for non Chrome/Firefox based browsers and, when that's the case, it also set the CORP policy.

Tested via coincident GitHub pages successfully on both GNOME Web and Epiphany TP.

This MR enables CORP implicitly for non Chrome/Firefox based browsers and, when that's the case, it also set the CORP policy.

Tested [via coincident GitHub pages](https://webreflection.github.io/coincident/test/hyperhtml.html) successfully on both GNOME Web and Epiphany TP.
@gzuidhof
Copy link
Owner

Perfect, thank you for this contribution!

@gzuidhof
Copy link
Owner

Could you perhaps

  • minify it (npm run minify)
  • bump the (minor) version number
  • update the README section?

I believe the README actually already includes some logic similar to this PR (confusingly)

@WebReflection
Copy link
Contributor Author

@gzuidhof it was a one-off contribution from GitHub site directly ... could you maybe do this after merge? Otherwise I'll apply your requested changes.

@gzuidhof
Copy link
Owner

Yeah I'm happy to do it :) It might be a while before I get around to it.. But I guess that's ok

@gzuidhof gzuidhof merged commit 0bd6c87 into gzuidhof:master Jul 9, 2023
@gzuidhof
Copy link
Owner

gzuidhof commented Jul 9, 2023

The changes are now merged and published in version 0.1.7

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