Add `/` behind all endpoints

actinia-org / actinia-core

Actinia Core is an open source REST API for scalable, distributed, high performance processing of geographical data that uses mainly GRASS GIS for computational tasks (DOI: https://doi.org/10.5281/zenodo.5879231) | Tutorial: https://actinia-org.github.io/actinia-core/ | Docker: https://hub.docker.com/r/mundialis/actinia-core

https://actinia.mundialis.de/

GNU General Public License v3.0

81 stars 23 forks source link

Add `/` behind all endpoints #246

Closed mmacata closed 3 years ago

mmacata commented 3 years ago

Currently a / behind the endpoint leads to a 404. Example:

https://actinia.mundialis.de/api/v1/version works
https://actinia.mundialis.de/api/v1/version/ leads to 404

Apparently this can be fixed so that both styles are recognized by adding a / behind the endpoint definition in endpoints.py

anikaweinmann commented 3 years ago

When I just tested it, it didn't work with curl though and the browser just kept appending / to the back of it.

juleshaas commented 3 years ago

Yes, the browser redirects .../endpoint to .../endpoint/, with curl however it only works with the / then. Would that still be an option?

mmacata commented 3 years ago

Then a wrapper for the endpoints or something like that would be nice. I don't think it is a good idea to change it if it doesn't work for both.