Page MenuHomePhabricator
Differential D15228

[e.cash] Enable CSP header
ClosedPublic
Actions

Authored by bytesofman on Jan 22 2024, 13:52.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC6ca2ccbd3bf0: [e.cash] Enable CSP header
Summary

T3395

We have this enabled as Report-Only right now. I'm not seeing any Report-Only log notices in browsing the site. So, actually enable it.

Test Plan

Navigate to https://e.cash and confirm you do not see any Content-Security-Police-Report-Only statements

Diff Detail

Repository
rABC Bitcoin ABC
Branch
enable-csp-home
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 26537
Build 52644: Build Diff
Build 52643: arc lint + arc unit

Event Timeline

bytesofman created this revision.Jan 22 2024, 13:52
Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 22 2024, 13:52
Harbormaster completed remote builds in B26537: Diff 44429.Jan 22 2024, 13:52
bytesofman published this revision for review.Jan 22 2024, 13:53
bytesofman edited the summary of this revision. (Show Details)
Fabien accepted this revision.Jan 22 2024, 14:17
This revision is now accepted and ready to land.Jan 22 2024, 14:17
Closed by commit rABC6ca2ccbd3bf0: [e.cash] Enable CSP header (authored by bytesofman). · Explain WhyJan 22 2024, 14:27
This revision was automatically updated to reflect the committed changes.
bytesofman added a commit: rABC6ca2ccbd3bf0: [e.cash] Enable CSP header.

Revision Contents
Changeset List

PathSize
web/
e.cash/
2 lines

Diff 44429

View Options

web/e.cash/next.config.js

Loading...