Search: unsafe use of subprocesses & curl

[amplifi]

Both search and search exports spawn subprocesses to execute curl. In search export, there's no explicit timeout applied to either the curl or the subprocess itself, no return code check, etc. These uses of subprocess.run() and curl should be replaced with a safer mechanism that properly handles the calls and results/cleanup; plenty of formal options exist for implementing the two use cases with a lower level of risk).

cadasta-platform/cadasta/search/export/resource.py

Line 55 in bc33ec4

subprocess.run([

cadasta-platform/cadasta/search/views/async.py

Line 281 in 665d710

subprocess.run([

[seav]

Since the search export feature has been removed, and if we put it back it should be an asynchronous process by then, I think we can close this issue.