mirror of
https://github.com/simple-login/app.git
synced 2024-09-27 20:31:30 +02:00
add "sub", fix test
This commit is contained in:
parent
3f7900f07e
commit
47109e6011
@ -435,7 +435,12 @@ class ClientUser(db.Model, ModelMixin):
|
|||||||
}
|
}
|
||||||
|
|
||||||
"""
|
"""
|
||||||
res = {"id": self.id, "client": self.client.name, "email_verified": True}
|
res = {
|
||||||
|
"id": self.id,
|
||||||
|
"client": self.client.name,
|
||||||
|
"email_verified": True,
|
||||||
|
"sub": str(self.id),
|
||||||
|
}
|
||||||
|
|
||||||
for scope in self.client.get_scopes():
|
for scope in self.client.get_scopes():
|
||||||
if scope == Scope.NAME:
|
if scope == Scope.NAME:
|
||||||
|
@ -189,8 +189,8 @@ def test_authorize_code_flow_no_openid_scope(flask_client):
|
|||||||
assert r.status_code == 200
|
assert r.status_code == 200
|
||||||
assert r.json["access_token"]
|
assert r.json["access_token"]
|
||||||
assert r.json["expires_in"] == 3600
|
assert r.json["expires_in"] == 3600
|
||||||
assert r.json["scope"] == ""
|
assert not r.json["scope"]
|
||||||
assert r.json["token_type"] == "bearer"
|
assert r.json["token_type"] == "Bearer"
|
||||||
|
|
||||||
assert r.json["user"] == {
|
assert r.json["user"] == {
|
||||||
"avatar_url": None,
|
"avatar_url": None,
|
||||||
@ -199,6 +199,7 @@ def test_authorize_code_flow_no_openid_scope(flask_client):
|
|||||||
"email_verified": True,
|
"email_verified": True,
|
||||||
"id": 1,
|
"id": 1,
|
||||||
"name": "AB CD",
|
"name": "AB CD",
|
||||||
|
"sub": "1",
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
@ -240,9 +241,9 @@ def test_authorize_code_flow_with_openid_scope(flask_client):
|
|||||||
assert not o.fragment
|
assert not o.fragment
|
||||||
|
|
||||||
# parse the query, should return something like
|
# parse the query, should return something like
|
||||||
# {'state': ['teststate'], 'code': ['knuyjepwvg']}
|
# {'state': ['teststate'], 'code': ['knuyjepwvg'], 'scope': ["openid"]}
|
||||||
queries = parse_qs(o.query)
|
queries = parse_qs(o.query)
|
||||||
assert len(queries) == 2
|
assert len(queries) == 3
|
||||||
|
|
||||||
assert queries["state"] == ["teststate"]
|
assert queries["state"] == ["teststate"]
|
||||||
assert len(queries["code"]) == 1
|
assert len(queries["code"]) == 1
|
||||||
@ -276,8 +277,8 @@ def test_authorize_code_flow_with_openid_scope(flask_client):
|
|||||||
assert r.status_code == 200
|
assert r.status_code == 200
|
||||||
assert r.json["access_token"]
|
assert r.json["access_token"]
|
||||||
assert r.json["expires_in"] == 3600
|
assert r.json["expires_in"] == 3600
|
||||||
assert r.json["scope"] == ""
|
assert r.json["scope"] == "openid"
|
||||||
assert r.json["token_type"] == "bearer"
|
assert r.json["token_type"] == "Bearer"
|
||||||
|
|
||||||
assert r.json["user"] == {
|
assert r.json["user"] == {
|
||||||
"avatar_url": None,
|
"avatar_url": None,
|
||||||
@ -286,6 +287,7 @@ def test_authorize_code_flow_with_openid_scope(flask_client):
|
|||||||
"email_verified": True,
|
"email_verified": True,
|
||||||
"id": 1,
|
"id": 1,
|
||||||
"name": "AB CD",
|
"name": "AB CD",
|
||||||
|
"sub": "1",
|
||||||
}
|
}
|
||||||
|
|
||||||
# id_token must be returned
|
# id_token must be returned
|
||||||
|
Loading…
Reference in New Issue
Block a user