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

Send search request as POST for large queries #701

Closed
zgoda-mobica opened this issue Oct 9, 2018 · 4 comments · Fixed by #1723
Closed

Send search request as POST for large queries #701

zgoda-mobica opened this issue Oct 9, 2018 · 4 comments · Fixed by #1723
Assignees

Comments

@zgoda-mobica
Copy link

Is your feature request related to a problem? Please describe.
Jira API allows search requests to be performed either as GET or POST, with POST suggested for queries that can't be encoded as query params. I have a query with ~60k clauses, ~60 bytes each after encoding as query param.

Describe the solution you'd like
For queries that can't be encoded as query params use POST request with JSON payload instead of GET.

@ssbarnea
Copy link
Member

ssbarnea commented Oct 9, 2018

I totally agree with your proposal. Now you only need to make a PR to address it.

@hdost
Copy link
Member

hdost commented Oct 16, 2018

There's even a PR currently pending that requires some changes #548

@stale
Copy link

stale bot commented Oct 15, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Oct 15, 2019
@ssbarnea ssbarnea removed the bounty label Oct 18, 2019
@stale stale bot removed the wontfix label Oct 18, 2019
@zgoda
Copy link

zgoda commented Nov 18, 2019

For over a year I used only POST requests for search in Jira and did not notice any issues. I think this should be default - it's hard to know what's the url size boundary and in theory this depends on servlet container implementation (can't say as I use only Tomcat).

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

Successfully merging a pull request may close this issue.

4 participants