Merge pull request 'update docs and comments' (#1) from testing into main
Reviewed-on: #1
This commit is contained in:
commit
7eb3ef7ce4
|
@ -10,4 +10,8 @@ See [the instructions](docs.md) for installing on a server.
|
||||||
|
|
||||||
## Testing
|
## Testing
|
||||||
|
|
||||||
There isn't a test suite, but you can use the `curl_test.sh` script to manually test that your app is working correctly.
|
There isn't a test suite, but you can use the `curl_test.sh` script to manually test that your app is working correctly.
|
||||||
|
|
||||||
|
## Notes for Hugh
|
||||||
|
|
||||||
|
Remember that you adjusted this on the server to run `process.js` to merge csv changes into the topojson file!
|
||||||
|
|
|
@ -51,7 +51,8 @@ async function gitPull(local_repo, res) {
|
||||||
app.post(`/${process.env.APP_PATH}`, (req, res) => {
|
app.post(`/${process.env.APP_PATH}`, (req, res) => {
|
||||||
const hmac = createHmac('sha256', process.env.SECRET)
|
const hmac = createHmac('sha256', process.env.SECRET)
|
||||||
const local_repo = process.env.LOCAL_REPO
|
const local_repo = process.env.LOCAL_REPO
|
||||||
hmac.update(JSON.stringify(req.body))
|
// Use spacing of 2 for Forgejo, or none for GitHub
|
||||||
|
hmac.update(JSON.stringify(req.body, null, 2))
|
||||||
|
|
||||||
// check has signature header and the decrypted signature matches
|
// check has signature header and the decrypted signature matches
|
||||||
if (req.get('X-Hub-Signature-256')) {
|
if (req.get('X-Hub-Signature-256')) {
|
||||||
|
@ -96,4 +97,4 @@ app.listen(port, () => {
|
||||||
}
|
}
|
||||||
|
|
||||||
console.log(`Webhooks app listening on port ${port}`)
|
console.log(`Webhooks app listening on port ${port}`)
|
||||||
})
|
})
|
||||||
|
|
Loading…
Reference in a new issue