PlutoDev/plutoapi-v2
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

API-16: Content-Type header required (#13)
All checks were successful
Jenkins Production Deployment
Details

Browse Source 
- Sending proper error message to the client when the content-type header is missing

Co-authored-by: Patrick Müller <patrick@mueller-patrick.tech>
Reviewed-on: #13
Co-authored-by: Patrick Müller <patrick@plutodev.de>
Co-committed-by: Patrick Müller <patrick@plutodev.de>
This commit is contained in:
Patrick Müller 2021-08-29 14:24:12 +00:00
parent 738004c7fe
commit a64a881598
2 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/models/partyplaner/login/Login.router.ts
View File

@ -17,6 +17,15 @@ loginRouter.post('/:isDevCall', async (req: Request, res: Response) => {
let password: string = '';
let useDev: boolean = (req.params.isDevCall ?? '') === 'dev'; // TBD
// Send error when content-type header is missing
if (!req.headers['content-type']) {
res.status(400).send({
'status': 'MISSING_CONTENT_TYPE',
'message': 'Please set the content-type header field'
});
return;
}
// API accepts both JSON in body and HTTP parameters
if (req.headers['content-type'] === 'application/json') {
username = req.body.username;

9
src/models/partyplaner/register/Register.router.ts
View File

@ -19,6 +19,15 @@ registerRouter.post('/:isDevCall', async (req: Request, res: Response) => {
let password: string = '';
let useDev: boolean = (req.params.isDevCall ?? '') === 'dev'; // TBD
// Send error when content-type header is missing
if (!req.headers['content-type']) {
res.status(400).send({
'status': 'MISSING_CONTENT_TYPE',
'message': 'Please set the content-type header field'
});
return;
}
// API accepts both JSON in body and HTTP parameters
if (req.headers['content-type'] === 'application/json') {
username = req.body.username;