OnroerendErfgoed / skosprovider_atramhasis

A skosprovider that can talk to an Atramhasis instance
http://skosprovider-atramhasis.readthedocs.org
MIT License
2 stars 0 forks source link

get_by_uri zou uri moeten urlencoden #69

Closed Wim-De-Clercq closed 4 years ago

Wim-De-Clercq commented 4 years ago

https://github.com/OnroerendErfgoed/skosprovider_atramhasis/blob/fc0900179c199afc90a0ae3883cd0278c573b013/skosprovider_atramhasis/providers.py#L65-L71

De code hier plakt de uri er gewoon aan. Maar omdat die slashes bevat is het veiliger om die te urlencoden. Vooral de // van https:// kan fout geïnterpreteerd worden door servers.

Verder gebruikt deze een deprecated api. En zou dit eigenlijk /uris?uri=xyz moeten doen ipv /uris/xyz