Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should /web and /search support GET requests? #66

Open
dhimmel opened this issue Dec 19, 2018 · 2 comments
Open

Should /web and /search support GET requests? #66

dhimmel opened this issue Dec 19, 2018 · 2 comments

Comments

@dhimmel
Copy link
Contributor

dhimmel commented Dec 19, 2018

We were curious regarding the reason to have the /web and /search endpoints encode the query as POST data rather than GET URL Parameters? The benefits of GET would be:

  1. Easier querying for casual users. For example, queries could be single URLs that would work in a browser as opposed to requiring curl.
  2. Easier caching via a reverse proxy. We are running tranlsation-server behind nginx and would like to cache repetitive queries, but caching responses based on POST data appears to be non-standard.

Would it be possible to support both POST data and GET URL parameters? Alternatively, is there a built-in caching solution to deal with 2?

@dhimmel
Copy link
Contributor Author

dhimmel commented Apr 18, 2019

@dstillman any thoughts on this?

Manubot now has a frontend developer on our team (@vincerubinetti), so it's possible we could contribute some develop efforts towards this issue.

@zotero zotero deleted a comment from dskandary Apr 28, 2019
@zotero zotero deleted a comment from dskandary Apr 28, 2019
@mvolz
Copy link
Contributor

mvolz commented Nov 24, 2021

You could try using citoid which does this and then queries translation-server, but running two services might be overkill for your use case :). https://www.mediawiki.org/wiki/Citoid/API

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants