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

Include user profile information in response #41

Closed
aaronpk opened this issue Jul 23, 2020 · 5 comments
Closed

Include user profile information in response #41

aaronpk opened this issue Jul 23, 2020 · 5 comments

Comments

@aaronpk
Copy link
Member

aaronpk commented Jul 23, 2020

There are some experiments of returning profile information in exchange for an authorization code in the response directly, simplifying the development of clients.

e.g.

{
  "me":"https://aaronparecki.com/",
  "profile": { 
    "type": "card",
    "name": "Aaron Parecki",
    "url": "https://aaronparecki.com/",
    "photo": "https://aaronparecki.com/images/profile.jpg"
  }
}

This can be returned whether or not an access token is also returned.

@dshanske
Copy link
Member

Related #31

@manton
Copy link

manton commented Aug 3, 2020

Micro.blog now includes profile too.

@tbhb
Copy link

tbhb commented Aug 8, 2020

Implemented now in Singulus for GET /indieauth/token, POST /indieauth/token, and POST /indieauth/authorize https://github.com/craftyphotons/singulus/commit/d1cf38ddb0bbad08d2240a00f10a077d00738d2c

@aaronpk
Copy link
Member Author

aaronpk commented Aug 8, 2020

This was discussed at the IndieAuth Popup Session, and the outcome of the discussion was:

  • General support for this feature
  • The profile data is not guaranteed to be verified in any way. It is what the user chooses to provide to the application for display purposes only. Definition of profile data and its authoritative nature has to be added to the spec.
  • Should be returned only when requested by the profile scope
  • Add email property and scope
  • Clarify the purpose of this which is informational to the application, not anything authoritative about the user and must not be used to identify users (e.g. making any decisions based on the email returned can lead to security holes)
  • Implementing this is optional
@aaronpk
Copy link
Member Author

aaronpk commented Sep 25, 2020

I'm starting to write the text for this, and realizing now that by not actually tying this vocabulary to jf2 or mf2, the type: card property is kind of redundant. The data here will always be from the h-card vocabulary anyway, so there doesn't seem to be any reason to include that property. Anyone opposed to just leaving that out since the IndieAuth spec is going to be defining the specific properties to be returned in the object anyway?

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