fix(e2e): set front_controller_active so Nextcloud generates clean URLs

Without Apache/mod_rewrite, Nextcloud redirects /apps/libresign to
/index.php/apps/libresign. The Vue Router then builds relative paths
from /index.php/, causing URL duplication and a broken app state.

PHP's built-in server reads the front_controller_active env variable
(see lib/private/URLGenerator.php). When true, Nextcloud generates
clean URLs without the index.php prefix — the same behavior that
Apache's .htaccess RewriteRule provides — and our router.php already
handles routing those clean URLs to index.php.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>

Author: vitormattos

.github/workflows/playwright.yml

@@ -188,28 +188,13 @@ jobs:
 
       - name: Start PHP built-in server
         run: |
-          php -S localhost:8080 -t . apps/${{ env.APP_NAME }}/playwright/router.php &
+          # front_controller_active tells Nextcloud to generate clean URLs (without index.php prefix)
+          # This mirrors what Apache mod_rewrite does via .htaccess RewriteRule
+          front_controller_active=true php -S localhost:8080 -t . apps/${{ env.APP_NAME }}/playwright/router.php &
           # Wait for server to become available
           timeout 30 bash -c 'until curl -s http://localhost:8080/status.php > /dev/null; do sleep 1; done'
           echo "Nextcloud is ready at http://localhost:8080"
 
-      - name: Debug - Check if build artifacts exist
-        run: |
-          echo "Checking for LibreSign JS build artifacts..."
-          ls -lah apps/${{ env.APP_NAME }}/js/ || echo "js/ directory not found"
-          echo "---"
-          echo "Testing status endpoint:"
-          curl -v http://localhost:8080/status.php
-          echo "---"
-          echo "Testing index endpoint:"
-          curl -I http://localhost:8080/index.php
-          echo "---"
-          echo "Testing app endpoint (should load HTML with JS):"
-          curl -s http://localhost:8080/apps/libresign | head -100
-          echo "---"
-          echo "Checking if app is enabled:"
-          php occ app:list | grep libresign
-
       - name: Install Playwright browsers
         working-directory: apps/${{ env.APP_NAME }}
         run: npx playwright install chromium --with-deps

playwright/e2e/sign-herself-with-click-to-sign.spec.ts

@@ -32,19 +32,7 @@ test('sign herself with click to sign', async ({ page }) => {
 		]),
 	)
 
-	// Capture console logs to debug
-	page.on('console', msg => console.log('BROWSER CONSOLE:', msg.type(), msg.text()))
-
-	// Debug: log the URL being constructed before goto
-	const testUrl = new URL('./apps/libresign', 'http://localhost:8080').toString()
-	console.log('Test URL constructed:', testUrl)
-	console.log('Page URL:', page.url())
-	console.log('Page title:', await page.title())
-	await page.screenshot({ path: 'debug-before-click.png', fullPage: true })
-	const bodyHTML = await page.evaluate(() => document.body.innerHTML)
-	console.log('Body HTML length:', bodyHTML.length)
-	console.log('Body HTML preview:', bodyHTML.substring(0, 500))
-
+	await page.goto('./apps/libresign')
 	await page.getByRole('button', { name: 'Upload from URL' }).click();
 	await page.getByRole('textbox', { name: 'URL of a PDF file' }).fill('https://raw.githubusercontent.com/LibreSign/libresign/main/tests/php/fixtures/pdfs/small_valid.pdf');
 	await page.getByRole('button', { name: 'Send' }).click();

playwright/router.php

@@ -16,15 +16,6 @@
 $uri = urldecode(parse_url($_SERVER['REQUEST_URI'], PHP_URL_PATH));
 $file = $rootDir . $uri;
 
-// Debug: log all requests (to stderr so it shows in PHP output)
-error_log(sprintf(
-	'[ROUTER] REQUEST_URI=%s | URI=%s | SCRIPT_NAME=%s | file_exists=%d',
-	$_SERVER['REQUEST_URI'],
-	$uri,
-	$_SERVER['SCRIPT_NAME'] ?? 'N/A',
-	file_exists($file) ? 1 : 0
-));
-
 // Serve static files as-is (except PHP files)
 if ($uri !== '/' && file_exists($file) && !is_dir($file) && pathinfo($uri, PATHINFO_EXTENSION) !== 'php') {
 	return false;
@@ -35,12 +26,6 @@
 	$_SERVER['SCRIPT_FILENAME'] = $rootDir . $script;
 	$_SERVER['PHP_SELF'] = $script;
 	$_SERVER['PATH_INFO'] = substr($uri, strlen($script)) ?: '';
-	error_log(sprintf(
-		'[ROUTER DISPATCH] SCRIPT=%s | PATH_INFO=%s | SCRIPT_FILENAME=%s',
-		$script,
-		$_SERVER['PATH_INFO'] ?? 'empty',
-		$_SERVER['SCRIPT_FILENAME']
-	));
 	require $rootDir . $script;
 };