Arraying / Arraying/Candor

[Dashboard] Security

Open
#8 3 comments 0 reactions 2 assignees Claimed by @martinmladenov View on GitHub
enhancement help wanted
Dominant language
TypeScript
Stars
8
Forks
0
PR merge metrics
No merged PRs in 30d

Description

The dashboard needs to be secure. The following endpoints need to be verified that they are hardened:
- [ ] `GET /api/pipelines` and `GET /api/pipelines/:pid` must only expose relevant information.
- [ ] `GET /api/pipelines/:pid/config` and `POST /api/pipelines/:pid/config` must only be accessible to assignees.
- [ ] `GET /api/runs/:pid/:rid/log` and `GET /api/runs/:pid/:rid/archived` are accessible only if the pipeline is public or the requester is assigned.
- [ ] `POST /trigger/:token` needs to be widely accessible.

Furthermore, CORS and cookie include needs to be set:
- [ ] When running in production, only allow same origin for `mode` and `cookies` in `fetch`.
- [ ] Otherwise, allow cross origin requests.

Lastly:
- [x] An instance of the dashboard should be tested against SSLLabs and receive an A+.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.