rit-sse / node-api

💯 The SSE New and Improved Unified API
https://sse.rit.edu/api/v2
MIT License
8 stars 9 forks source link

Can't delete go links containing '/' #94

Closed BenAlderfer closed 4 years ago

BenAlderfer commented 4 years ago

Links like the one in the screenshot throw a 404 when you try to delete them image

gavrielrh commented 4 years ago

Honestly think we might just want to prevent go links with a slash in the name. A slash implies that there's a directory, where there most definitely isn't for that go link.

We could also potentially set up the api to work via post bodies, eg:

[DELETE] https://sse.rit.edu/api/v2/links
{
  shortLink: "hbbq/signin"
}

which would handle this sort of thing.

Any ideas? I definitely agree we should resolve this 😄