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

Make nonce-bearing Abaco URIs cleaner via URL rewriting #54

Open
mwvaughn opened this issue Dec 4, 2018 · 0 comments
Open

Make nonce-bearing Abaco URIs cleaner via URL rewriting #54

mwvaughn opened this issue Dec 4, 2018 · 0 comments
Assignees
Labels
enhancement priority.verylow Don't even think about starting work on this

Comments

@mwvaughn
Copy link

mwvaughn commented Dec 4, 2018

Here's an example callback used in some ongoing work that leverages Abaco, where actor EEDKw7NAr4E0x can accept this message and use it to set a value for a variable shorthashin database record 1073f4ff-c2b9-5190-bd9a-e6a406d9796a.

https://api.sd2e.org/actors/v2/EEDKw7NAr4E0x/messages?x-nonce=TACC_kOMDBMNGo1r3m&shorthash=3f643e7b2722f16e&uuid=1073f4ff-c2b9-5190-bd9a-e6a406d9796a

Compared to the callback URL generated by several popular platforms, this feels a bit clunky and I think it is because the extended string of URL parameters carries both the nonce and the payload parameters.

It feels more intuitive for the base URL to include the nonce inline:

https://api.sd2e.org/actors/v2/EEDKw7NAr4E0x/messages/x-nonce/TACC_kOMDBMNGo1r3m

This makes it very clear which part of the URL is user payload versus the portion authorizing access to the actor.

https://api.sd2e.org/actors/v2/EEDKw7NAr4E0x/messages/x-nonce/TACC_kOMDBMNGo1r3m?token=3f643e7b2722f16e&uuid=1073f4ff-c2b9-5190-bd9a-e6a406d9796a

@mwvaughn mwvaughn added enhancement priority.verylow Don't even think about starting work on this labels Dec 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement priority.verylow Don't even think about starting work on this
Projects
None yet
Development

No branches or pull requests

2 participants