Move JWT config to Bedrock env config for consistency

artshumrc · Sep 6, 2022 · 98c6e22 · 98c6e22
1 parent 9395f48
commit 98c6e22
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/bedrock/config/application.php b/bedrock/config/application.php
@@ -113,6 +113,11 @@
 Config::define('SCRIPT_DEBUG', false);
 ini_set('display_errors', '0');
 
+/**
+ * JWT Authentication
+ */
+Config::define('JWT_AUTH_SECRET_KEY', env('JWT_AUTH_SECRET_KEY'));
+
 /**
  * Allow WordPress to detect HTTPS when used behind a reverse proxy or a load balancer
  * See https://codex.wordpress.org/Function_Reference/is_ssl#Notes

diff --git a/bedrock/web/app/themes/orpheus/functions.php b/bedrock/web/app/themes/orpheus/functions.php
@@ -570,5 +570,5 @@ function twentyseventeen_front_page_template( $template ) {
 
 add_filter( 'allowed_http_origin', '__return_true' );
 
-define('JWT_AUTH_SECRET_KEY', 'quidfaciatlaetassegetes');
+// define('JWT_AUTH_SECRET_KEY', 'quidfaciatlaetassegetes'); // Moved to Bedrock config, should not be set in a visible way
 define('JWT_AUTH_CORS_ENABLE', false);