Paths

Table of Contentst

Differential D14639

[alias-server] Use helmet to improve express security
ClosedPublic
Actions

Authored by bytesofman on Oct 13 2023, 15:45.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCacda88e6a693: [alias-server] Use helmet to improve express security

Summary

Per express docs security improvement recommendations, implement helmet in alias-server

Ref https://expressjs.com/en/advanced/best-practice-security.html
Ref https://www.npmjs.com/package/helmet

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bytesofman created this revision.Oct 13 2023, 15:45

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 13 2023, 15:45

bytesofman requested review of this revision.Oct 13 2023, 15:45

Harbormaster completed remote builds in B25349: Diff 42668.Oct 13 2023, 15:46

PiRK accepted this revision.Oct 16 2023, 06:19

This revision is now accepted and ready to land.Oct 16 2023, 06:19

Closed by commit rABCacda88e6a693: [alias-server] Use helmet to improve express security (authored by bytesofman). · Explain WhyOct 16 2023, 13:17

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCacda88e6a693: [alias-server] Use helmet to improve express security.

Revision Contents
Changeset List

Path

Size

apps/

alias-server/

package-lock.json

14 lines

package.json

1 line

src/

app.js

2 lines

Diff 42693

View Options

apps/alias-server/package-lock.json

View Options

apps/alias-server/package.json

View Options

apps/alias-server/src/app.js

[alias-server] Use helmet to improve express securityClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 42693

apps/alias-server/package-lock.json

apps/alias-server/package.json

apps/alias-server/src/app.js

[alias-server] Use helmet to improve express security
ClosedPublic
Actions

Revision Contents
Changeset List