openwisp-config icon indicating copy to clipboard operation
openwisp-config copied to clipboard

Allow url parameter to be a list

Open nemesifier opened this issue 9 years ago • 4 comments

A list would be very handy in order to allow more URLs, in case one fails.

nemesifier avatar Feb 22 '16 18:02 nemesifier

I think this is rather load balancer responsibility to handle failover

sheerun avatar May 19 '16 20:05 sheerun

I've forked openwisp-config to https://github.com/BlancLoup/openwisp-config I've added redundant urls with backward compatibility, but have some issue to resolve: Only way to check url connection is curl to "curl -Is --connect-timeout 10 -k -X POST https://server/controller/register/" that returns header "X-Openwisp-Controller: true". It's hack, not solution. Perfect: url something like "https://server/controller/heartbeat/" on controller's side. NB: i have not tested yet this modification

BlancLoup avatar May 02 '18 19:05 BlancLoup

@BlancLoup could you open a pull request marked as WIP (work in progress) please? That way it will be easier to review and track.

Remember to reference this issue by adding #15 in the description of the pull request.

nemesifier avatar May 03 '18 20:05 nemesifier

@nemesisdesign can you review my pull request cause i have little experience with git?

BlancLoup avatar May 03 '18 21:05 BlancLoup