fix in accesso multi-tenancy

Author: madbob

code/app/Helpers/Paths.php

@@ -1,10 +1,12 @@
 <?php
 
-/*
-    Se la variabile d'ambiente STORAGE_FOLDER è definita, tutti i files vengono
-    conservati nella cartella (contenuta in storage/) con quel nome.
-    Utile per isolare i files in configurazioni multi-GAS
-*/
+use Illuminate\Support\Env;
+
+/**
+ * Se la variabile d'ambiente STORAGE_FOLDER è definita, tutti i files vengono
+ * conservati nella cartella (contenuta in storage/) con quel nome.
+ * Utile per isolare i files in configurazioni multi-GAS
+ */
 function gas_storage_path($path = null, $folder = false)
 {
     $ret = storage_path();
@@ -25,22 +27,22 @@ function gas_storage_path($path = null, $folder = false)
     return $ret;
 }
 
-/*
-    Questa funzione restituisce il file .env relativo all'istanza.-
-    In caso di una installazione standalone non ci sono particolari problemi: il
-    file .env è .env
-    In caso di una installazione in multi-tenancy, il file dipende dal dominio
-    HTTP che viene richiesto. Questo comportamento viene sfruttato in
-    bootstrap/app.php per inizializzare l'istanza ad ogni richiesta HTTP.
-    Se invece sto eseguendo uno script CLI (ad esempio: i comandi eseguiti in
-    cron), e non posso accedere alla variabile $_SERVER (non esistendo alcuna
-    richiesta HTTP da leggere), i possibili comportamenti sono due: o appunto ho
-    appena eseguito il comando, dunque non è ancora stato inizializzato niente,
-    dunque prendo per buono sempre e solo .env, altrimenti desumo l'istanza
-    locale dal valore di APP_URL (che nel frattempo è stato letto dal file .env
-    giusto)
-*/
-function env_file()
+/**
+ * Questa funzione restituisce il file .env relativo all'istanza.-
+ * In caso di una installazione standalone non ci sono particolari problemi: il
+ * file .env è .env
+ * In caso di una installazione in multi-tenancy, il file dipende dal dominio
+ * HTTP che viene richiesto. Questo comportamento viene sfruttato in
+ * bootstrap/app.php per inizializzare l'istanza ad ogni richiesta HTTP.
+ * Se invece sto eseguendo uno script CLI (ad esempio: i comandi eseguiti in
+ * cron), e non posso accedere alla variabile $_SERVER (non esistendo alcuna
+ * richiesta HTTP da leggere), i possibili comportamenti sono due: o appunto ho
+ * appena eseguito il comando, dunque non è ancora stato inizializzato niente,
+ * dunque prendo per buono sempre e solo .env, altrimenti desumo l'istanza
+ * locale dal valore di APP_URL (che nel frattempo è stato letto dal file .env
+ * giusto)
+ */
+function envFile()
 {
     if (global_multi_installation()) {
         $instance = null;
@@ -49,7 +51,13 @@ function env_file()
             $instance = substr($_SERVER['HTTP_HOST'], 0, strpos($_SERVER['HTTP_HOST'], '.'));
         }
         elseif (app()->runningInConsole()) {
-            $domain = parse_url(config('app.url'), PHP_URL_HOST);
+            /*
+                Reminder: quando questa funzione viene invocata, il service
+                provider che gestisce le configurazioni (normalmente acceduto
+                con l'helper config()) non è ancora stato inizializzato.
+                Non cadere nella tentazione di usare qui config('app.url')
+            */
+            $domain = parse_url(Env::get('APP_URL'), PHP_URL_HOST);
             $instance = preg_replace('/^([^\.]*)\.gasdotto\.net.*$/', '\1', $domain);
         }
 

code/bootstrap/app.php

@@ -20,7 +20,7 @@
     .env (nominato in funzione del dominio)
 */
 if (global_multi_installation()) {
-    $env_file = env_file();
+    $env_file = envFile();
     if (file_exists(base_path() . '/' . $env_file) == false) {
         exit('Non sembra esistere nessuna istanza con questo nome. Visita <a href="http://gasdotto.net">gasdotto.net</a> per registrarne gratuitamente una!');
     }