Page MenuHomePhorge
Differential D4082

Exclude horizon routes from CSP headers
ClosedPublic
Actions

Authored by mollekopf on Feb 26 2023, 2:36 PM.
Tags
None
Referenced Files
F16562772: D4082.id11699.diff
Wed, Oct 30, 1:49 AM
Unknown Object (File)
Sun, Oct 27, 10:32 AM
Unknown Object (File)
Sun, Oct 27, 3:21 AM
Unknown Object (File)
Thu, Oct 24, 7:48 PM
Unknown Object (File)
Mon, Oct 21, 1:49 AM
Unknown Object (File)
Thu, Oct 17, 11:51 AM
Unknown Object (File)
Thu, Oct 17, 7:04 AM
Unknown Object (File)
Tue, Oct 15, 7:25 AM
View All 98 Files
Subscribers
None

Details

Reviewers
machniak
Group Reviewers
Restricted Project
Commits
rK185f0d12509d: Exclude horizon routes from CSP headers
rKcc74495240c6: Exclude horizon routes from CSP headers
rK51dfc947ccec: Exclude horizon routes from CSP headers
Summary

For me CSP breaks horizon, and according to the github issue it is expected that horizon is only used internally and no CSP headers are applied.

Diff Detail

Repository
rK kolab
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mollekopf requested review of this revision.Feb 26 2023, 2:36 PM
mollekopf created this revision.
Harbormaster completed remote builds in B41690: Diff 11699.Feb 26 2023, 2:36 PM
mollekopf edited the summary of this revision. (Show Details)Feb 26 2023, 2:37 PM
mollekopf added a reviewer: Restricted Project.
machniak accepted this revision.Feb 27 2023, 2:52 PM
This revision is now accepted and ready to land.Feb 27 2023, 2:52 PM
Closed by commit rK185f0d12509d: Exclude horizon routes from CSP headers (authored by mollekopf). · Explain WhyMar 4 2023, 2:27 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
app/
Http/
Middleware/
5 lines

Diff 11771

View Options

src/app/Http/Middleware/ContentSecurityPolicy.php

Loading...