wrapped request refresh jwt hook with memoize

Author: yujin-wu

dash/dash-renderer/src/AppContainer.react.js

@@ -6,7 +6,7 @@ import Loading from './components/core/Loading.react';
 import Toolbar from './components/core/Toolbar.react';
 import Reloader from './components/core/Reloader.react';
 import {setHooks, setConfig} from './actions/index';
-import {type} from 'ramda';
+import {type, memoizeWith, identity} from 'ramda';
 
 class UnconnectedAppContainer extends React.Component {
     constructor(props) {
@@ -16,6 +16,12 @@ class UnconnectedAppContainer extends React.Component {
             props.hooks.request_post !== null ||
             props.hooks.request_refresh_jwt !== null
         ) {
+            if (props.hooks.request_refresh_jwt) {
+                props.hooks.request_refresh_jwt = memoizeWith(
+                    identity,
+                    props.hooks.request_refresh_jwt
+                );
+            }
             props.dispatch(setHooks(props.hooks));
         }
     }

tests/integration/renderer/test_request_hooks.py

@@ -189,6 +189,7 @@ def update_output(value):
 
     dash_duo.percy_snapshot(name="request-hooks interpolated")
 
+
 def test_rdrh003_refresh_jwt(dash_duo):
 
     app = Dash(__name__)
@@ -225,18 +226,10 @@ def test_rdrh003_refresh_jwt(dash_duo):
         </body>
     </html>"""
 
-
     app.layout = html.Div(
         [
             dcc.Input(id="input", value="initial value"),
-            html.Div(
-                html.Div(
-                    [
-                        html.Div(id="output-1"),
-                        html.Div(id="output-token")
-                    ]
-                )
-            ),
+            html.Div(html.Div([html.Div(id="output-1"), html.Div(id="output-token")])),
         ]
     )
 
@@ -251,18 +244,21 @@ def protect_route(func):
         @functools.wraps(func)
         def wrap(*args, **kwargs):
             try:
-                if flask.request.method == 'OPTIONS':
+                if flask.request.method == "OPTIONS":
                     return func(*args, **kwargs)
-                token = flask.request.authorization or flask.request.headers.environ.get(
-                    "HTTP_AUTHORIZATION"
+                token = (
+                    flask.request.authorization
+                    or flask.request.headers.environ.get("HTTP_AUTHORIZATION")
                 )
-                if required_jwt_len and (not token or len(token) != required_jwt_len + len('Bearer ')):
+                if required_jwt_len and (
+                    not token or len(token) != required_jwt_len + len("Bearer ")
+                ):
                     flask.abort(401, description="JWT Expired " + str(token))
             except HTTPException as e:
                 return e
             return func(*args, **kwargs)
-        return wrap
 
+        return wrap
 
     # wrap all API calls with auth.
     for name, method in (