samvera / questioning_authority

Question your authorities
Other
54 stars 30 forks source link

Build better SPARQL for ULAN queries with spaces in them #178

Closed mbklein closed 5 years ago

mbklein commented 5 years ago

Getty's server was choking on the previous multi-term SPARQL, possibly because of a URI encoding error. This PR changes the way multi-term queries are built and encoded. Also updates specs to match.

Fixes #163

Note: There is some other misbehaving SPARQL, notably in the TGN authority searcher but possibly others as well. We are focusing on a ULAN fix because that's what is hampering our production rollout, but hopefully this can be used as an example for working with the others.