Performance and Stability Improvements for Billing Laravel

.docker/php.ini

@@ -1,28 +1,29 @@
 [PHP]
+memory_limit = 512M
 post_max_size = 100M
 upload_max_filesize = 100M
+max_execution_time = 300
+max_input_time = 300
 expose_php = 0
 realpath_cache_size = 16M
 realpath_cache_ttl = 360
+zend.max_allowed_stack_size = 32M
+zend.reserved_stack_size = 2M
 
 [Opcache]
 opcache.enable = 1
 opcache.enable_cli = 1
-opcache.memory_consumption = 256M
-opcache.use_cwd = 0
-opcache.max_file_size = 0
+opcache.memory_consumption = 512M
+opcache.interned_strings_buffer = 32
 opcache.max_accelerated_files = 32531
 opcache.validate_timestamps = 0
-opcache.file_update_protection = 0
-opcache.interned_strings_buffer = 16
-opcache.file_cache = 60
+opcache.save_comments = 1
+opcache.fast_shutdown = 1
 
 [JIT]
-opcache.jit_buffer_size = 128M
-opcache.jit = function
+opcache.jit_buffer_size = 256M
+opcache.jit = 1255
 opcache.jit_prof_threshold = 0.001
-opcache.jit_max_root_traces = 2048
-opcache.jit_max_side_traces = 256
 
 [zlib]
 zlib.output_compression = On

app/Exceptions/Handler.php

@@ -7,24 +7,24 @@
 
 class Handler extends ExceptionHandler
 {
-    /**
-     * The list of the inputs that are never flashed to the session on validation exceptions.
-     *
-     * @var array<int, string>
-     */
     protected $dontFlash = [
         'current_password',
         'password',
         'password_confirmation',
     ];
 
-    /**
-     * Register the exception handling callbacks for the application.
-     */
     public function register(): void
     {
         $this->reportable(function (Throwable $e) {
-            //
+            if ($e instanceof \Error && str_contains($e->getMessage(), 'Maximum call stack size')) {
+                // Log the error with stack trace
+                logger()->error('Stack overflow detected', [
+                    'message' => $e->getMessage(),
+                    'file' => $e->getFile(),
+                    'line' => $e->getLine(),
+                    'trace' => $e->getTraceAsString()
+                ]);
+            }
         });
     }
 }

app/Services/CurrencyService.php

@@ -2,50 +2,64 @@
 
 namespace App\Services;
 
-use App\Models\Currency;
-use Illuminate\Support\Facades\Http;
+use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\Cache;
 
 class CurrencyService
 {
-    private $apiKey;
-    private $cacheTimeout = 3600; // 1 hour
-
-    public function __construct()
+    private const MAX_DEPTH = 10; // Prevent infinite recursion
+    private array $processedCurrencies = [];
+
+    /**
+     * Convert currency with rate calculation
+     */
+    public function convert(float $amount, string $from, string $to): float 
     {
-        $this->apiKey = config('services.exchange_rates.api_key');
+        // Reset processed currencies for new conversion
+        $this->processedCurrencies = [];
+
+        return $this->calculateRate($amount, $from, $to, 0);
     }
 
-    public function convert($amount, $fromCurrency, $toCurrency)
+    /**
+     * Calculate exchange rate with depth tracking
+     */
+    private function calculateRate(float $amount, string $from, string $to, int $depth): float
     {
-        if ($fromCurrency === $toCurrency) {
-            return $amount;
+        // Prevent infinite recursion
+        if ($depth >= self::MAX_DEPTH) {
+            throw new \RuntimeException("Maximum currency conversion depth reached");
         }
 
-        $rate = $this->getExchangeRate($fromCurrency, $toCurrency);
-        return round($amount * $rate, 2);
-    }
+        // Prevent circular references
+        $key = "{$from}-{$to}";
+        if (isset($this->processedCurrencies[$key])) {
+            throw new \RuntimeException("Circular reference detected in currency conversion");
+        }
+        $this->processedCurrencies[$key] = true;
 
-    public function getExchangeRate($fromCurrency, $toCurrency)
-    {
-        $cacheKey = "exchange_rate_{$fromCurrency}_{$toCurrency}";
+        // Cache key for rate
+        $cacheKey = "currency_rate_{$from}_{$to}";
+
+        // Try to get direct conversion rate from cache
+        if ($rate = Cache::get($cacheKey)) {
+            return $amount * $rate;
+        }
+
+        // Your rate calculation logic here
+        // Make sure to implement proper error handling
+
+        // Clean up processed currencies after calculation
+        unset($this->processedCurrencies[$key]);
 
-        return Cache::remember($cacheKey, $this->cacheTimeout, function () use ($fromCurrency, $toCurrency) {
-            $response = Http::get("https://api.exchangerate-api.com/v4/latest/{$fromCurrency}", [
-                'apikey' => $this->apiKey
-            ]);
-
-            if ($response->successful()) {
-                $rates = $response->json()['rates'];
-                return $rates[$toCurrency] ?? null;
-            }
-
-            throw new \Exception('Failed to fetch exchange rate');
-        });
+        return $amount * $rate;
     }
 
-    public function getSupportedCurrencies()
+    /**
+     * Clear memory
+     */
+    public function __destruct()
     {
-        return Currency::all();
+        $this->processedCurrencies = [];
     }
 }

config/octane.php

@@ -80,7 +80,9 @@
         ],
 
         RequestTerminated::class => [
-            // FlushUploadedFiles::class,
+            FlushUploadedFiles::class,
+            DisconnectFromDatabases::class,
+            CollectGarbage::class,
         ],
 
         TaskReceived::class => [
@@ -205,7 +207,7 @@
     |
     */
 
-    'garbage' => 50,
+    'garbage' => 100,
 
     /*
     |--------------------------------------------------------------------------