Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 7 additions & 4 deletions resources/views/components/layout.blade.php
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,13 @@
x-bind:class="{ 'dark': isDark === true }"
>
<head>
<meta
http-equiv="Content-Security-Policy"
content="upgrade-insecure-requests"
/>
{{-- Skipped locally so the site can be served over plain http --}}
@unless (app()->isLocal())
<meta
http-equiv="Content-Security-Policy"
content="upgrade-insecure-requests"
/>
@endunless
<meta charset="utf-8" />
<meta
http-equiv="X-UA-Compatible"
Expand Down
5 changes: 4 additions & 1 deletion resources/views/components/layouts/auth.blade.php
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,10 @@
x-bind:class="{ 'dark': isDark === true }"
>
<head>
<meta http-equiv="Content-Security-Policy" content="upgrade-insecure-requests" />
{{-- Skipped locally so the site can be served over plain http --}}
@unless (app()->isLocal())
<meta http-equiv="Content-Security-Policy" content="upgrade-insecure-requests" />
@endunless
<meta charset="utf-8" />
<meta http-equiv="X-UA-Compatible" content="IE=edge" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
Expand Down
11 changes: 7 additions & 4 deletions resources/views/components/layouts/dashboard.blade.php
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,13 @@
x-bind:class="{ 'dark': isDark === true }"
>
<head>
<meta
http-equiv="Content-Security-Policy"
content="upgrade-insecure-requests"
/>
{{-- Skipped locally so the site can be served over plain http --}}
@unless (app()->isLocal())
<meta
http-equiv="Content-Security-Policy"
content="upgrade-insecure-requests"
/>
@endunless
<meta charset="utf-8" />
<meta
http-equiv="X-UA-Compatible"
Expand Down
63 changes: 63 additions & 0 deletions tests/Feature/UpgradeInsecureRequestsTest.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,63 @@
<?php

namespace Tests\Feature;

use App\Features\ShowAuthButtons;
use App\Models\User;
use Illuminate\Foundation\Testing\RefreshDatabase;
use Laravel\Pennant\Feature;
use PHPUnit\Framework\Attributes\DataProvider;
use PHPUnit\Framework\Attributes\Test;
use Tests\TestCase;

class UpgradeInsecureRequestsTest extends TestCase
{
use RefreshDatabase;

protected function setUp(): void
{
parent::setUp();

Feature::define(ShowAuthButtons::class, true);
}

public static function pages(): array
{
return [
'main layout' => ['/docs/mobile/3/getting-started/introduction', false],
'auth layout' => ['/login', false],
'dashboard layout' => ['/dashboard', true],
];
}

#[Test]
#[DataProvider('pages')]
public function pages_upgrade_insecure_requests_outside_local(string $uri, bool $signedIn): void
{
$this->app['env'] = 'production';

$this->getPage($uri, $signedIn)
->assertStatus(200)
->assertSee('content="upgrade-insecure-requests"', false);
}

#[Test]
#[DataProvider('pages')]
public function pages_do_not_upgrade_insecure_requests_locally(string $uri, bool $signedIn): void
{
$this->app['env'] = 'local';

$this->getPage($uri, $signedIn)
->assertStatus(200)
->assertDontSee('upgrade-insecure-requests', false);
}

private function getPage(string $uri, bool $signedIn)
{
if ($signedIn) {
$this->actingAs(User::factory()->create());
}

return $this->withoutVite()->get($uri);
}
}
Loading