dpgaspar commented on a change in pull request #17984:
URL: https://github.com/apache/superset/pull/17984#discussion_r781953373
##########
File path: superset/initialization/__init__.py
##########
@@ -572,12 +573,27 @@ def init_app_in_ctx(self) -> None:
self.init_views()
+ def check_secret_key(self) -> None:
+ if self.config["SECRET_KEY"] == CHANGE_ME_SECRET_KEY:
+ top_banner = 80 * "-" + "\n" + 36 * " " + "WARNING\n" + 80 * "-"
+ bottom_banner = 80 * "-" + "\n" + 80 * "-"
+ logger.warning(top_banner)
+ logger.warning(
+ "A Default SECRET_KEY was detected please use
superset_config.py "
+ "to override it.\n"
+ "Use a strong complex alphanumeric string and use a tool to
help"
+ " you generate \n"
+ "a sufficiently random sequence, ex: openssl rand -base64 42"
+ )
+ logger.warning(bottom_banner)
Review comment:
I evaluated something similar, but I'm avoiding break pip installations
and docker. It's really nice to have a really fast frictionless first
installations.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org
