Closed lokeoke closed 3 months ago
Currently https://github.com/mblackgeo/flask-cognito-lib/blob/main/src/flask_cognito_lib/decorators.py#L21 instantiates a new Config() instead of using one from the app context. This results in an issue where user wants to override the Config class but it only applies to https://github.com/mblackgeo/flask-cognito-lib/blob/main/src/flask_cognito_lib/plugin.py.
Config()
Config
In my specific use-case instead of using AWS_COGNITO_REDIRECT_URL and AWS_COGNITO_LOGOUT_URL I want to generate them using url_for(e.g. return url_for(endpoint='auth.cognito', _external=True)) and it isn't possible due to the issue.
AWS_COGNITO_REDIRECT_URL
AWS_COGNITO_LOGOUT_URL
url_for
return url_for(endpoint='auth.cognito', _external=True)
Closed by #43
mblackgeo
commented
3 months ago
Currently https://github.com/mblackgeo/flask-cognito-lib/blob/main/src/flask_cognito_lib/decorators.py#L21 instantiates a new
Config()instead of using one from the app context. This results in an issue where user wants to override theConfigclass but it only applies to https://github.com/mblackgeo/flask-cognito-lib/blob/main/src/flask_cognito_lib/plugin.py.In my specific use-case instead of using
AWS_COGNITO_REDIRECT_URLandAWS_COGNITO_LOGOUT_URLI want to generate them usingurl_for(e.g.return url_for(endpoint='auth.cognito', _external=True)) and it isn't possible due to the issue.