• assigned: fredm, zachs
• priority: urgent
• type: bug
• status: closed, completed
• keywords: collections

Go to the following correlation results:

• https://genenetwork.org/corr_compute?job_id=9922450b-3d68-40e3-bfa2-33e0056e8cca

• Click the "More Options" button
• Filter the 500 entries by r>0, r<1, mean>3 and mean<100.
• Click the "Select Traits" button to activate your filter. This leaves only 73 entries.
• Select all of traits except the two in chromosome 16 which means 71 entries were selected.
• Click "+ Add" button to create new collection
• A new window opens and the error message "414 Request-URI Too Large" shows up

This issue is also reproducible on https://cd.genenetwork.org

The request uses the "GET" method: see

• https://github.com/genenetwork/genenetwork2/blob/3ebf796f82797139b68f29e1e6a2323d60886539/wqflask/wqflask/collect.py#L118-L119

Some files that are concerned with the issue:

• https://github.com/genenetwork/genenetwork2/blob/3ebf796f82797139b68f29e1e6a2323d60886539/wqflask/wqflask/collect.py#L101-L108

• https://github.com/genenetwork/genenetwork2/blob/3ebf796f82797139b68f29e1e6a2323d60886539/wqflask/wqflask/templates/collections/add.html#L8

• https://github.com/genenetwork/genenetwork2/blob/3ebf796f82797139b68f29e1e6a2323d60886539/wqflask/wqflask/static/new/javascript/search_results.js#L192-L198

Probable solution: add `method="POST"` to the "<form ...>" line in the "add.html" file, but there is a chance that might break other features that depend on the same form. This will need to be done carefully.

In retrospect, this feature should not use "GET" at all, since it is not indempotent. Every run has the potential to change the state of the underlying collection(s). All features depending on the "add.html" template should use the "POST" method, never "GET".

• https://github.com/genenetwork/genenetwork2/commit/1915fcdf7f67cd76348b06070e86aedfb4076f01

This converts the requests to use the POST method which does not have any data size limits according to the HTTP protocol (in practice, there is probably a limit, but that is a configuration value, and we are unlikely to hit it).