Skip to content

Commit dccb6c4

Browse files
committed
chore: improve /repo error handling
1 parent b808b61 commit dccb6c4

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

src/service/routes/repo.js

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ router.delete('/:name/user/push/:username', async (req, res) => {
110110
});
111111

112112
router.delete('/:name/delete', async (req, res) => {
113-
if (req.user.admin) {
113+
if (req.user && req.user.admin) {
114114
const repoName = req.params.name;
115115

116116
await db.deleteRepo(repoName);
@@ -124,6 +124,13 @@ router.delete('/:name/delete', async (req, res) => {
124124

125125
router.post('/', async (req, res) => {
126126
if (req.user && req.user.admin) {
127+
if (!req.body.name) {
128+
res.status(400).send({
129+
message: 'Repository name is required',
130+
});
131+
return;
132+
}
133+
127134
const repo = await db.getRepo(req.body.name);
128135
if (repo) {
129136
res.status(409).send({

0 commit comments

Comments
 (0)