Updated #75 removed nested if and avoid using keys() as suggested

Author: reiven

examples/simple.py

@@ -13,19 +13,20 @@
 # function is used to actually generate the token
 @app.route('/login', methods=['POST'])
 def login():
-    if request.is_json:
-        params = request.get_json()
-        if 'username' in params.keys() and 'password' in params.keys():
-            if params['username'] != 'test' or params['password'] != 'test':
-                return jsonify({"msg": "Bad username or password"}), 401
-        else:
-            return jsonify({"msg": "Missing auth parameters"}), 401
-
-        # Identity can be any data that is json serializable
-        ret = {'access_token': create_access_token(identity=params['username'])}
-        return jsonify(ret), 200
-    else:
+    if not request.is_json:
         return jsonify({"msg": "Missing auth"}), 401
+    params = request.get_json()
+    if 'username' in params and 'password' in params:
+        if params['username'] != 'test' or params['password'] != 'test':
+            return jsonify({"msg": "Bad username or password"}), 401
+    else:
+        return jsonify({"msg": "Missing auth parameters"}), 401
+
+    # Identity can be any data that is json serializable
+    ret = {'access_token': create_access_token(identity=params['username'])}
+    return jsonify(ret), 200
+
+        
 
 # Protect a view with jwt_required, which requires a valid access token
 # in the request to access.