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
Unknown Object (File)
Mon, Aug 26, 1:07 PM
Unknown Object (File)
Tue, Aug 13, 2:13 PM
Unknown Object (File)
Jul 30 2024, 10:21 AM
Unknown Object (File)
Jul 6 2024, 9:41 AM
Unknown Object (File)
Jul 1 2024, 8:18 AM
Unknown Object (File)
Jun 28 2024, 2:59 PM
Unknown Object (File)
Jun 8 2024, 6:31 AM
Unknown Object (File)
Jun 7 2024, 3:38 PM
View All 77 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 Passed
Unit
No Test Coverage
Build Status
Buildable 41690
Build 16733: arc lint + arc unit

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 11699

View Options

src/app/Http/Middleware/ContentSecurityPolicy.php

Loading...