Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Commit adc6eab

Browse files
committed
MSC2918: use parse_boolean to get query parameter value
Signed-off-by: Quentin Gliech <quentingliech@gmail.com>
1 parent e402a07 commit adc6eab

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

synapse/rest/client/v1/login.py

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919

2020
from typing_extensions import TypedDict
2121

22+
import synapse.http.servlet
2223
from synapse.api.errors import Codes, LoginError, SynapseError
2324
from synapse.api.ratelimiting import Ratelimiter
2425
from synapse.api.urls import CLIENT_API_PREFIX
@@ -30,6 +31,7 @@
3031
RestServlet,
3132
parse_json_object_from_request,
3233
parse_string,
34+
parse_boolean,
3335
)
3436
from synapse.http.site import SynapseRequest
3537
from synapse.rest.client.v2_alpha._base import client_patterns
@@ -158,12 +160,7 @@ def on_GET(self, request: SynapseRequest):
158160
async def on_POST(self, request: SynapseRequest):
159161
login_submission = parse_json_object_from_request(request)
160162

161-
param = bytes(LoginRestServlet.REFRESH_TOKEN_PARAM, "utf-8")
162-
should_issue_refresh_token = (
163-
request.args is not None
164-
and param in request.args
165-
and request.args[param][0] == b"true"
166-
)
163+
should_issue_refresh_token = parse_boolean(request, name=LoginRestServlet.REFRESH_TOKEN_PARAM, default=False)
167164

168165
try:
169166
if login_submission["type"] == LoginRestServlet.APPSERVICE_TYPE:

0 commit comments

Comments
 (0)