Add alternative means of changing the URL generation

Author: niladam

readme.md

@@ -411,6 +411,28 @@ class CustomExport extends ExcelExport
 }
 ```
 
+## File download URL customization
+
+By default, the package generates a signed URL with a default expiration time of 24 hours. 
+The URL contains the filename including the extension. Some WAF (Web Application Firewall) solutions can block the URL due to the fact that it links to a file, which contains parameters and can cause a false positive.
+
+```php
+// Somewhere in a ServiceProvider in the `boot()` method.
+use pxlrbt\FilamentExcel\FilamentExport;
+
+FilamentExport::createExportUrlUsing(function ($export) {
+    $fileInfo = pathinfo($export['filename']);
+    $filenameWithoutExtension = $fileInfo['filename'];
+    $extension = $fileInfo['extension'];
+
+    return URL::temporarySignedRoute(
+        'your-custom-route',
+        now()->addHours(2),
+        ['path' => $filenameWithoutExtension, 'extension' => $extension]
+    );
+});
+```
+
 ## Contributing
 
 If you want to contribute to this packages, you may want to test it in a real Filament project:

src/FilamentExcelServiceProvider.php

@@ -3,12 +3,8 @@
 namespace pxlrbt\FilamentExcel;
 
 use Filament\Facades\Filament;
-use Filament\Notifications\Actions\Action;
-use Filament\Notifications\Notification;
 use Illuminate\Console\Scheduling\Schedule;
 use Illuminate\Support\Facades\Event;
-use Illuminate\Support\Facades\Storage;
-use Illuminate\Support\Facades\URL;
 use Illuminate\Support\Str;
 use pxlrbt\FilamentExcel\Commands\PruneExportsCommand;
 use pxlrbt\FilamentExcel\Events\ExportFinishedEvent;
@@ -38,7 +34,7 @@ public function configurePackage(Package $package): void
 
     public function bootingPackage()
     {
-        Filament::serving($this->sendExportFinishedNotification(...));
+        Filament::serving(fn () => app(FilamentExport::class)->sendNotification());
 
         $this->callAfterResolving(Schedule::class, function (Schedule $schedule) {
             $schedule->command(PruneExportsCommand::class)->daily();
@@ -47,65 +43,13 @@ public function bootingPackage()
         Event::listen(ExportFinishedEvent::class, [$this, 'cacheExportFinishedNotification']);
     }
 
-    public function sendExportFinishedNotification(): void
-    {
-        $exports = cache()->pull($this->getNotificationCacheKey(auth()->id()));
-
-        if (! filled($exports)) {
-            return;
-        }
-
-        foreach ($exports as $export) {
-            $url = URL::temporarySignedRoute(
-                'filament-excel-download',
-                now()->addHours(24),
-                ['path' => $export['filename']]
-            );
-
-            if (! Storage::disk('filament-excel')->exists($export['filename'])) {
-                continue;
-            }
-
-            if (Filament::getCurrentPanel()->hasDatabaseNotifications()) {
-                Notification::make(data_get($export, 'id'))
-                    ->title(__('filament-excel::notifications.download_ready.title'))
-                    ->body(__('filament-excel::notifications.download_ready.body'))
-                    ->success()
-                    ->icon('heroicon-o-arrow-down-tray')
-                    ->actions([
-                        Action::make('download')
-                            ->label(__('filament-excel::notifications.download_ready.download'))
-                            ->url($url, shouldOpenInNewTab: true)
-                            ->button()
-                            ->close(),
-                    ])
-                    ->sendToDatabase(auth()->user());
-            } else {
-                Notification::make(data_get($export, 'id'))
-                    ->title(__('filament-excel::notifications.download_ready.title'))
-                    ->body(__('filament-excel::notifications.download_ready.body'))
-                    ->success()
-                    ->icon('heroicon-o-arrow-down-tray')
-                    ->actions([
-                        Action::make('download')
-                            ->label(__('filament-excel::notifications.download_ready.download'))
-                            ->url($url, shouldOpenInNewTab: true)
-                            ->button()
-                            ->close(),
-                    ])
-                    ->persistent()
-                    ->send();
-            }
-        }
-    }
-
     public function cacheExportFinishedNotification(ExportFinishedEvent $event): void
     {
         if ($event->userId === null) {
             return;
         }
 
-        $key = $this->getNotificationCacheKey($event->userId);
+        $key = FilamentExport::getNotificationCacheKey($event->userId);
 
         $exports = cache()->pull($key, []);
         $exports[] = [
@@ -116,9 +60,4 @@ public function cacheExportFinishedNotification(ExportFinishedEvent $event): voi
 
         cache()->put($key, $exports);
     }
-
-    protected function getNotificationCacheKey($userId): string
-    {
-        return 'filament-excel:exports:'.$userId;
-    }
 }

src/FilamentExport.php

@@ -0,0 +1,96 @@
+<?php
+
+namespace pxlrbt\FilamentExcel;
+
+use Closure;
+use Filament\Facades\Filament;
+use Filament\Notifications\Actions\Action;
+use Filament\Notifications\Notification;
+use Illuminate\Support\Facades\Storage;
+use Illuminate\Support\Facades\URL;
+
+class FilamentExport
+{
+    public static ?Closure $createExportUrlUsing = null;
+
+    public static function createExportUrlUsing(Closure $closure): void
+    {
+        static::$createExportUrlUsing = $closure;
+    }
+
+    protected function sendDatabaseNotification(array $export, string $url): void
+    {
+        Notification::make(data_get($export, 'id'))
+            ->title(__('filament-excel::notifications.download_ready.title'))
+            ->body(__('filament-excel::notifications.download_ready.body'))
+            ->success()
+            ->icon('heroicon-o-arrow-down-tray')
+            ->actions([
+                Action::make('download')
+                    ->label(__('filament-excel::notifications.download_ready.download'))
+                    ->url($url, shouldOpenInNewTab: true)
+                    ->button()
+                    ->close(),
+            ])
+            ->sendToDatabase(auth()->user());
+    }
+
+    protected function sendPersistentNotification(array $export, string $url): void
+    {
+        Notification::make(data_get($export, 'id'))
+            ->title(__('filament-excel::notifications.download_ready.title'))
+            ->body(__('filament-excel::notifications.download_ready.body'))
+            ->success()
+            ->icon('heroicon-o-arrow-down-tray')
+            ->actions([
+                Action::make('download')
+                    ->label(__('filament-excel::notifications.download_ready.download'))
+                    ->url($url, shouldOpenInNewTab: true)
+                    ->button()
+                    ->close(),
+            ])
+            ->persistent()
+            ->send();
+    }
+
+    public function sendNotification(): void
+    {
+        $exports = cache()->pull(static::getNotificationCacheKey(auth()->id()));
+
+        if (! filled($exports)) {
+            return;
+        }
+
+        foreach ($exports as $export) {
+            $url = $this->createUrl($export);
+
+            if (! Storage::disk('filament-excel')->exists($export['filename'])) {
+                continue;
+            }
+
+            if (Filament::getCurrentPanel()->hasDatabaseNotifications()) {
+                $this->sendDatabaseNotification($export, $url);
+            } else {
+                $this->sendPersistentNotification($export, $url);
+            }
+        }
+    }
+
+    public static function getNotificationCacheKey($userId): string
+    {
+        return 'filament-excel:exports:'.$userId;
+    }
+
+    protected function createUrl(array $export): string
+    {
+        if (is_null(static::$createExportUrlUsing)) {
+            return URL::temporarySignedRoute(
+                'filament-excel-download',
+                now()->addHours(24),
+                ['path' => $export['filename']]
+            );
+        }
+
+        return call_user_func(static::$createExportUrlUsing, $export);
+    }
+}