search

PathwayCommons / cpath2

Biological pathway data integration and access platform (Pathway Commons)
http://www.pathwaycommons.org/pc2/
MIT License
6 stars 5 forks source link

Disable HTTP POST requests (in PC v11 and after) #300

Closed IgorRodchenkov closed 5 years ago

IgorRodchenkov commented 6 years ago

Currently, in Pathway Commons core web services, POST request mapping:

  1. makes proxying and, more important, caching (thus performance) problematic (requires configuring additional cache for server-side java methods, which has been there in PC9 disabled anyway);
  2. is not required, in fact (the data is static once the PC is released; frankly, POST was added simply to allow for submitting large arrays or values (URIs or IDs) with /get, /graph, /traverse queryies, - a feature which has hardly been used in practice ever since, nor it makes much sense nor practical);
  3. wrong, for a POST request should update information or execute a command over http, which is not quite the case in our current system (though years ago, we did have an idea to submit pathway data or analysis code to be merged/executed on the server..; also, running/executing a graph traversal query still means reading/getting the information rather than modifying.)

Thsi change requires updating (or - better - removing) the cpath-client module and CyPath2 Cytoscape app.

IgorRodchenkov commented 5 years ago

Well, alright, won't do this. Let's support POST as well.