Move index() inline html into its template

rayluo · rayluo · commit fd0572647060 · 2019-09-24T12:30:32.000-07:00
diff --git a/app.py b/app.py
@@ -15,10 +15,7 @@
 def index():
     if not session.get("user"):
         return redirect(url_for("login"))
-    return """Welcome, %s.
-        <li><a href='/graphcall'>Call Microsoft Graph API</a></li>
-        <li><a href="/logout">Logout</a></li>
-        """ % session["user"].get("name")
+    return render_template('index.html', user=session["user"])
 
 @app.route("/login")
 def login():
@@ -48,7 +45,7 @@ def authorized():
 
 @app.route("/logout")
 def logout():
-    session["user"] = None  # Mark current session as not-logged-in
+    session["user"] = None  # Log out from this app from its session
     # session.clear()  # If you prefer, this would nuke the user's token cache too
     return redirect(  # Also need to logout from Microsoft Identity platform
         "https://login.microsoftonline.com/common/oauth2/v2.0/logout"
diff --git a/templates/index.html b/templates/index.html
@@ -0,0 +1,13 @@
+<!DOCTYPE html>
+<html lang="en">
+<head>
+    <meta charset="UTF-8">
+</head>
+<body>
+    <h1>Microsoft Identity Python Web App</h1>
+    Welcome {{ user.get("name") }}!
+    <li><a href='/graphcall'>Call Microsoft Graph API</a></li>
+    <li><a href="/logout">Logout</a></li>
+</body>
+</html>
+
