Skip to content
This repository has been archived by the owner on May 27, 2024. It is now read-only.

Frontend Geocoder Failover Infinite Loop #356

Closed
jmfield2 opened this issue Oct 3, 2016 · 0 comments
Closed

Frontend Geocoder Failover Infinite Loop #356

jmfield2 opened this issue Oct 3, 2016 · 0 comments
Assignees
Labels

Comments

@jmfield2
Copy link
Contributor

jmfield2 commented Oct 3, 2016

For a limited degree of failover, we define two geocoder URLs in the config.js (one matches the current hostname (maps), and the last is hard-coded as mobullity. On error, the geocoder will find the "next" URL in the list and try that one. This seemed to work OK, but on mobullity, that means if the mobullity geocoder has an error, the failover loop will never end and continuously hit the mobullity geocoder endpoint until the browser is closed before looking for the mobullity URL will always return the "next" value to try.

One solution that I worked on is to use index values - starting at 0 - and incrementing on error until it reaches the end of the list (currently 2). I also added an alert message if it reaches the end so that the user can be aware of a problem instead of waiting indefinitely for results that would never come.

@jmfield2 jmfield2 added the bug label Oct 3, 2016
@jmfield2 jmfield2 self-assigned this Oct 3, 2016
@jmfield2 jmfield2 modified the milestone: Sprint 10-7-16 Oct 7, 2016
jmfield2 added a commit that referenced this issue Oct 8, 2016
Fix #356 - Change URL failover code to use numeric indexes
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant