-
-
Notifications
You must be signed in to change notification settings - Fork 183
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
Expose WebSocket for network access #696
Conversation
874e467
to
703904a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, since we're adding WebAssembly as a default endowment, we should get rid of the endowment permission for it. We should still add XMLHttpRequest
to the network access endowment.
Edit: While we're here, it might be worth looking into whether we can add WebSocket
as one of the available APIs.
703904a
to
4929b40
Compare
Done!
The only issue with adding |
According to this implementation https://github.com/heineiuo/isomorphic-ws you can use the browser WebSocket and
I guess we could strip out some of these things in the same way we do for WebAssembly? 🤔 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wait, scratch that. XMLHttpRequest
doesn't exist in Node, and we should probably remove it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
* Expose XMLHttpRequest for network access endowment * Add WebSocket * Fix linting * Remove XmlHttpRequest since its not available in Node.js
* Expose XMLHttpRequest for network access endowment * Add WebSocket * Fix linting * Remove XmlHttpRequest since its not available in Node.js
* Expose XMLHttpRequest for network access endowment * Add WebSocket * Fix linting * Remove XmlHttpRequest since its not available in Node.js
Expose
WebSocket
for network access endowment.Description
WebSocket
for network access endowment.Checklist