Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow url parameter to be a list #15

Open
nemesifier opened this issue Feb 22, 2016 · 4 comments
Open

Allow url parameter to be a list #15

nemesifier opened this issue Feb 22, 2016 · 4 comments
Assignees

Comments

@nemesifier
Copy link
Member

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

@nemesifier nemesifier self-assigned this Feb 22, 2016
@nemesifier nemesifier modified the milestones: 0.3, 0.4 Feb 22, 2016
@nemesifier nemesifier changed the title New feature: allow url parameter to be a list Allow url parameter to be a list Mar 3, 2016
@nemesifier nemesifier removed this from the 0.4 milestone May 18, 2016
@sheerun
Copy link

sheerun commented May 19, 2016

I think this is rather load balancer responsibility to handle failover

@nemesifier nemesifier moved this from To do (general) to To do (OpenWRT/LEDE) in OpenWISP Contributor's Board Apr 29, 2018
@BlancLoup
Copy link

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

@nemesifier
Copy link
Member Author

@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.

@BlancLoup
Copy link

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

@nemesifier nemesifier moved this from To do (OpenWRT/LEDE) to To do (general) in OpenWISP Contributor's Board May 4, 2018
@nemesifier nemesifier moved this from To do (general) to In progress in OpenWISP Contributor's Board May 4, 2018
@nemesifier nemesifier removed this from In progress in OpenWISP Contributor's Board May 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants