From 89afa440856c92e3bb0b57936a40e50da65df5f1 Mon Sep 17 00:00:00 2001 From: Frederick Muriuki Muriithi Date: Mon, 9 Oct 2023 06:51:44 +0300 Subject: issue: Update issue with troubleshooting notes. --- issues/add-to-collection-error.gmi | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/issues/add-to-collection-error.gmi b/issues/add-to-collection-error.gmi index e131eb2..09de0ae 100644 --- a/issues/add-to-collection-error.gmi +++ b/issues/add-to-collection-error.gmi @@ -37,3 +37,21 @@ There is a chance the link above might eventually expire. If that happens, you c * Set "Location Type" = "Gene" * Click "Compute" * Continue with the steps in the section above + + +## Troubleshooting Notes + +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 "
" 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". -- cgit v1.2.3