Make $form['#action'] = 'dispatch-data/run_analysis' configurable in instruments

[dsavchenko]

Needed for frontend installations, which asks different dispatchers

[volodymyrss]

should we ask @motame to do this, what do you think?

[dsavchenko]

It's easy enough so that anyone of us can do it. But as @motame should review anyway, probably he can just do it quickly.

[motame]

A new config parameter is added: data_server_local_url.
'data_server_local_url' => 'dispatch-data/run_analysis'
Note that it's better to consider only local paths (web server aliases).
If an external provider is needed, a proxy should configured in the web server configuration e.g.
ProxyPass /mmoda/dispatch-data/ https://frontend-prod.obsuks1.unige.ch/mmoda/dispatch-data/

[dsavchenko]

Small concern about naming: it was meant to be used with external dispatcher, so local could be misleading.
On the other hand, if you advise using proxy for this (why?) then the name is ok (meaning local vs global)

[volodymyrss]

I guess this name is meant to be rather "relative" than local?

[motame]

local here is for "local web server path/alias" which can be indeed an alias to a local service (less probable in our case) or a proxypass to an external url as shown above.
And Indeed, @dsavchenko, I advise using proxypass to avoid issues with CORS, Cross-Origin Resource Sharing.
Allowing the page to make XMLHttpRequest directly from external services may raise security issues !

[dsavchenko]

I didn't think about CORS. Thanks !

[volodymyrss]

is this over @dsavchenko ?

[dsavchenko]

is this over @dsavchenko ?

Seems that. I didn't try it “in the wild”, though.

[motame]

Yes it is !
And pushed to github reference