Only do this when getting a config or the application requests discovery. When getting the server list this should maybe not be called, but eduvpn-common might need to have a separate function e.g. "get_delisted_servers" that returns which current servers are delisted. This way the server list can load as fast as possible and the application can asynchronously update the delisted servers with a warning symbol
jwijenbergh
commented
1 year ago
When register is called discovery MUST NOT be called. https://github.com/eduvpn/eduvpn-common/blob/main/client/client.go#L197-L204
Only do this when getting a config or the application requests discovery. When getting the server list this should maybe not be called, but eduvpn-common might need to have a separate function e.g. "get_delisted_servers" that returns which current servers are delisted. This way the server list can load as fast as possible and the application can asynchronously update the delisted servers with a warning symbol