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

Choosing between javascript and http communication. #131

Closed
wants to merge 2 commits into from

Conversation

tommythorsen
Copy link
Member

This changes the spec proposal so that a payment app can decide whether
it wants to communicate with the user agent by means of HTTP or by means
of javascript functions.

The choice is implemented as an additional manifest field called
communication_method.

See issue #130

This changes the spec proposal so that a payment app can decide whether
it wants to communicate with the user agent by means of HTTP or by means
of javascript functions.

The choice is implemented as an additional manifest field called
communication_method.
@dlongley
Copy link
Contributor

dlongley commented May 18, 2016

Maybe the app should declare its supported protocols instead and let the mediator decide which is best for its platform/context.

@ianbjacobs
Copy link
Contributor

+1 to merging the PR (mostly to review it; not ready to endorse the proposal though yet).

Thanks!

Ian

This lets the payment app declare which communication methods it
supports so that the user agent can choose which method to use.
@tommythorsen
Copy link
Member Author

@dlongley

Maybe the app should declare its supported protocols instead and let the mediator decide which is best for its platform/context.

This sounds like a good idea. I pushed another commit that does this.

@adrianhopebailie
Copy link
Collaborator

Based on discussion in the TF we want to pick one way to do this and demonstrate that it is easy to acheive the other through a simple bridge (i.e. if we use HTTP then a JavaScript service worker can provide an offline JavaScript implementation and if we use JS then it can send messages to a remote service)

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.

4 participants