OpenRefine
OpenRefine copied to clipboard
Drop JSONP support for reconciliation APIs
Users should not be allowed to give remote services injection/JSONP capabilities given the danger. Today someone might just add a malicious service for a popular database to the testbench and only provide JSONP support and then all users trying it will be compromised without warning.
Proposed solution
Only three services on the test bench which do not support CORS supports JSONP so I suggest we drop support.
Alternatives considered
#7185 would allow us to keep supporting non-CORS enabled services while dropping JSONP. I personally don't think we should wait.
Additional context
I count 6 services, the most prominent of which is VIVO, but it's still something we should consider.