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

Clarifications to authentication mechanisms #371

Merged
merged 1 commit into from Aug 15, 2016

Conversation

Projects
None yet
3 participants
@richvdh
Member

richvdh commented Aug 15, 2016

  • Make the purpose of the auth key in /register requests explicit, and say
    that it should be empty at first.
  • Restructure the UA-auth section a bit.
  • In the UA-auth section, say that clients should submit no auth to start
    with, and add 'Stage 0' representing this to the example.
  • s/{stage,login} type/authentication type/ in the UA-auth section. Seems
    clearer to me.
  • Try to distinguish the example responses from the example requests by giving
    an HTTP header.

Fixes https://matrix.org/jira/browse/SPEC-441.

Clarifications to authentication mechanisms
* Make the purpose of the `auth` key in /register requests explicit, and say
  that it should be empty at first.

* Restructure the UA-auth section a bit.

* In the UA-auth section, say that clients should submit no `auth` to start
  with, and add 'Stage 0' representing  this to the example.

* s/{stage,login} type/authentication type/ in the UA-auth section. Seems
   clearer to me.

* Try to distinguish the example responses from the example requests by giving
  an HTTP header.
@richvdh

This comment has been minimized.

Member

richvdh commented Aug 15, 2016

@erikjohnston: do you know about auth?

@erikjohnston

This comment has been minimized.

Member

erikjohnston commented Aug 15, 2016

No, I don't really. @dbkr ?

@erikjohnston erikjohnston assigned dbkr and unassigned erikjohnston Aug 15, 2016

@dbkr dbkr merged commit c494adf into master Aug 15, 2016

2 checks passed

Docs (Commit) Build #1757 succeeded in 36 sec
Details
Docs (Merged PR) Build finished.
Details

@richvdh richvdh deleted the rav/auth_clarifications branch Aug 15, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment