Skip to content

Commit d724baa

Browse files
committed
Recover from mismatched state to index page, not login page
1 parent 5a8b881 commit d724baa

1 file changed

Lines changed: 11 additions & 12 deletions

File tree

app.py

Lines changed: 11 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -29,18 +29,17 @@ def login():
2929

3030
@app.route(app_config.REDIRECT_PATH) # Its absolute URL must match your app's redirect_uri set in AAD
3131
def authorized():
32-
if request.args['state'] != session.get("state"):
33-
return redirect(url_for("login"))
34-
cache = _load_cache()
35-
result = _build_msal_app(cache=cache).acquire_token_by_authorization_code(
36-
request.args['code'],
37-
scopes=app_config.SCOPE, # Misspelled scope would cause an HTTP 400 error here
38-
redirect_uri=url_for("authorized", _external=True))
39-
if "error" in result:
40-
return "Login failure: %s, %s" % (
41-
result["error"], result.get("error_description"))
42-
session["user"] = result.get("id_token_claims")
43-
_save_cache(cache)
32+
if request.args.get('state') == session.get("state"):
33+
cache = _load_cache()
34+
result = _build_msal_app(cache=cache).acquire_token_by_authorization_code(
35+
request.args['code'],
36+
scopes=app_config.SCOPE, # Misspelled scope would cause an HTTP 400 error here
37+
redirect_uri=url_for("authorized", _external=True))
38+
if "error" in result:
39+
return "Login failure: %s, %s" % (
40+
result["error"], result.get("error_description"))
41+
session["user"] = result.get("id_token_claims")
42+
_save_cache(cache)
4443
return redirect(url_for("index"))
4544

4645
@app.route("/logout")

0 commit comments

Comments
 (0)