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

SEP-45: Fix server auth entry verification #1630

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions ecosystem/sep-0045.md
Original file line number Diff line number Diff line change
Expand Up @@ -103,12 +103,11 @@ The authentication flow is as follows:
1. The **Server** verifies that the `function_name` in each authorization entry is `web_auth_verify`
1. The **Server** verifies that the `args` map in each authorization entry match the expected values and are the same
across all authorization entries:
1. The `account` value matches the **Client Account** address
1. The `home_domain` value matches the **Home Domain**
1. The `home_domain_address` value matches the **Home Domain Address**
1. The `web_auth_domain` value matches the **Server**'s domain
1. The `client_domain_address` value matches the **Client Domain Address** if the **Client** included a
`client_domain` in the request, otherwise it is not present
1. The `client_domain` is present if `client_domain_address` is present
1. The `client_domain_address` value matches the **Client Domain Address** if `client_domain` is present
1. (Optional) The **Server** verifies that the `nonce` argument is the same across all authorization entries and is
unique
1. The **Server** verifies that there is an authorization entry where `credentials.address.address` is the **Home Domain
Expand Down
Loading