Allow POST for queries

Question

Allow POST for queries

alexisdimi opened this issue 5 years ago · comments

Alexis Dimitriadis commented 5 years ago

A sparql query can easily get too large to fit in a GET request; SPARQLWrapper can be configured to do POST requests, but sparql-kernel does not seem to support them. Can this feature be added?

Paulo Villegas · Answer 1 · Mon Jul 08 2019 05:48:25 GMT+0800 (China Standard Time)

Yes, that is easy. I've added it in the 1.3.0 branch, currently in testing, via a %method magic. As soon as it is tested, I will generate a new release.

Paulo Villegas · Answer 2 · Mon Jul 08 2019 12:53:31 GMT+0800 (China Standard Time)

I've seen you have submitted a PR on this. Thanks!

Alexis Dimitriadis · Answer 3 · Wed Jul 10 2019 17:06:51 GMT+0800 (China Standard Time)

You got a PR for this changeset, that implements POST queries? Yeah I did it myself because I needed it, it was pretty easy after all. I didn't realize I had submitted the PR, though.

Alexis Dimitriadis · Answer 4 · Wed Jul 10 2019 17:42:23 GMT+0800 (China Standard Time)

I see the changeset got appended to the other, irrelevant PR... Clearly I don't know my way around this github feature yet! Anyway the patch has been working fine so far.

Paulo Villegas · Answer 5 · Wed Jul 10 2019 22:27:55 GMT+0800 (China Standard Time)

Yes, it was included in the other PR, that's why I only noticed it later. Your patch is virtually identical to the solution in my branch, so it will work the same (there's not much space for imagination here, anyway :-)