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

Do not send me to callback #10

Open
sebsel opened this issue Jun 29, 2017 · 4 comments
Open

Do not send me to callback #10

sebsel opened this issue Jun 29, 2017 · 4 comments
Labels

Comments

@sebsel
Copy link
Collaborator

sebsel commented Jun 29, 2017

Like in aaronpk/IndieAuth.com#85, the me parameter should not be send to the callback. (Consumers should use the state param.)

See also https://chat.indieweb.org/dev/2017-06-30#t1498778310000000 :)

@Zegnat
Copy link
Collaborator

Zegnat commented Jun 30, 2017

While I just changed how the redirect URL is built (e62ad9f), I did not remove the me parameter as it is still in the reference flow. Anyone going by that flow might expect to get it back. Until developer documentation is updated I think it is best to keep it as is.

@dissolve
Copy link

sounds like a good idea, lets keep this open until that is done.

@aaronpk
Copy link
Contributor

aaronpk commented Jul 1, 2017

It's going to take a while to deprecate this parameter since many clients are still expecting it. We should track down as many Micropub clients as we can and open issues like this on all of them: aaronpk/Quill#79

@dissolve
Copy link

dissolve commented Jul 2, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants