search

betterbettor / bet-better-server

RESTful API for BetBetter
MIT License
0 stars 0 forks source link

Feat/sec #13

Closed krissolui closed 8 months ago

krissolui commented 8 months ago

Description

  1. Installed helmet as dependency
  2. Use helmet middleware to secure Express app by setting HTTP response headers
  3. Remove unnecessary use of body-parser

Screenshots

Above: before using helmet. Below: after using helmet

Screenshot 2023-10-16 at 17 24 46

Passed snyk code scan

Screenshot 2023-10-16 at 17 24 57
louvrecly commented 8 months ago

@krissolui Sorry, I've used Rebase and merge instead of Create a merge commit in the previous PRs. That's why there are conflicts here. You may need to rebase the PR to the latest main branch before merging this one. 

// checkout to the main branch and pull the latest changes
git checkout main
git pull origin main

// checkout to the feat/sec branch and rebase it from the latest main branch
git checkout feat/sec
git rebase main

// resolve the conflicts if any then force push it to remote
git push origin feat/sec -f
krissolui commented 8 months ago

@louvrecly ok. will merge #14 first.