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

Recommend RPs to store transport hints #1299

Merged
merged 4 commits into from Oct 9, 2019
Merged

Conversation

emlun
Copy link
Member

@emlun emlun commented Sep 11, 2019

Fixes #1188, and also formalises the beginning of the RP registration operation as a prerequisite. The same formalisation is also applied to the authentication operation, although unnecessary, for symmetry.


Preview | Diff

@emlun emlun added this to the L2-WD-02 milestone Sep 11, 2019
@emlun emlun self-assigned this Sep 11, 2019
Copy link
Contributor

@equalsJeffH equalsJeffH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM overall @emlun, thx! some modest comments below.

index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
@equalsJeffH equalsJeffH self-requested a review October 2, 2019 22:55
Copy link
Contributor

@equalsJeffH equalsJeffH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, let's merge this, thx @emlun!

@emlun emlun merged commit 72e2f47 into master Oct 9, 2019
@emlun emlun deleted the issue-1188-rp-transports branch October 9, 2019 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

finish up on "provide transport info during registration"
3 participants