Skip to content

Commit dd06312

Browse files
committed
mirage: Add authentication checks to yank/unyank route handlers
1 parent c94e845 commit dd06312

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

mirage/route-handlers/crates.js

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -265,6 +265,11 @@ export function register(server) {
265265
});
266266

267267
server.delete('/api/v1/crates/:name/:version/yank', (schema, request) => {
268+
let { user } = getSession(schema);
269+
if (!user) {
270+
return new Response(403, {}, { errors: [{ detail: 'must be logged in to perform that action' }] });
271+
}
272+
268273
const { name, version: versionNum } = request.params;
269274
const crate = schema.crates.findBy({ name });
270275
if (!crate) {
@@ -282,6 +287,11 @@ export function register(server) {
282287
});
283288

284289
server.put('/api/v1/crates/:name/:version/unyank', (schema, request) => {
290+
let { user } = getSession(schema);
291+
if (!user) {
292+
return new Response(403, {}, { errors: [{ detail: 'must be logged in to perform that action' }] });
293+
}
294+
285295
const { name, version: versionNum } = request.params;
286296
const crate = schema.crates.findBy({ name });
287297
if (!crate) {

0 commit comments

Comments
 (0)