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 channel support #5

Open
lawnsea opened this issue Aug 22, 2013 · 1 comment
Open

Add channel support #5

lawnsea opened this issue Aug 22, 2013 · 1 comment

Comments

@lawnsea
Copy link
Contributor

lawnsea commented Aug 22, 2013

It would be good to provide an easy way for postMessage clients and servers to
ignore messages not intended for them. The right way to do this is natively with
MessageChannels, but FF doesn't support those and the polyfill from tilde adds a
lot of code we don't want in the TCB.[1]

So, if the config includes a channel value when the client/server is
constructed, include that value as channel when txing and ignore rxed messages
that do not have an identical channel value.

[1] We could use the same technique that we do for tiny-jsonrpc-postmessage,
which is to just implement the protocol w/o the library support in the trusted
code, but that may be complicated.

@lawnsea
Copy link
Contributor Author

lawnsea commented Aug 23, 2013

MessageChannel.js is sweet, but
sends events encoded with kamino, which
is too complicated for use by the trusted portion of the broker.

So, we'll do channels at the protocol level, instead of the transport level.

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

No branches or pull requests

1 participant