Skip to content

Remove duplicate proxy middleware - #4632

Merged
jbrooksuk merged 1 commit into
3.xfrom
codex/fix-trusted-proxies
Aug 11, 2026
Merged

Remove duplicate proxy middleware#4632
jbrooksuk merged 1 commit into
3.xfrom
codex/fix-trusted-proxies

Conversation

@jbrooksuk

Copy link
Copy Markdown
Member

Summary

  • remove Cachet's application-specific trusted-proxy wrapper
  • rely on Cachet Core to configure Laravel's built-in middleware

Why

Core now owns the CACHET_TRUSTED_PROXIES configuration and preserves Laravel's * wildcard semantics. Removing this wrapper prevents the application from converting * into ['*'].

Validation

  • php artisan test

@jbrooksuk
jbrooksuk marked this pull request as ready for review August 11, 2026 18:55
@jbrooksuk
jbrooksuk merged commit 679188c into 3.x Aug 11, 2026
6 of 7 checks passed
@jbrooksuk
jbrooksuk deleted the codex/fix-trusted-proxies branch August 11, 2026 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant