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

ProtonVPN servers missing after update #2332

Closed
virtitnerd opened this issue Jun 22, 2024 · 8 comments
Closed

ProtonVPN servers missing after update #2332

virtitnerd opened this issue Jun 22, 2024 · 8 comments

Comments

@virtitnerd
Copy link

The current server list for ProtonVPN is missing a large amount of Server's currently available to users. API is public to view current server listing with country, location, domain, load, etc.

ProtonVPN API : https://api.protonmail.ch/vpn/logicals

When ProtonVPN is selected, could it be configured to use the api then to select the best server based on the load if we specify the SERVER_COUNTRIES and SERVER_CITIES variables?

Copy link
Contributor

@qdm12 is more or less the only maintainer of this project and works on it in his free time.
Please:

@qdm12
Copy link
Owner

qdm12 commented Jul 8, 2024

The current server list for ProtonVPN is missing a large amount of Server's currently available to users

Update your servers, check out https://github.com/qdm12/gluetun-wiki/blob/main/setup/servers.md#update-the-vpn-servers-list

When ProtonVPN is selected, could it be configured to use the api then to select the best server based on the load if we specify the SERVER_COUNTRIES and SERVER_CITIES variables?

Not for the time being, see https://github.com/qdm12/gluetun-wiki/blob/main/faq/firewall.md#timing

@qdm12 qdm12 closed this as not planned Won't fix, can't repro, duplicate, stale Jul 8, 2024
Copy link
Contributor

github-actions bot commented Jul 8, 2024

Closed issues are NOT monitored, so commenting here is likely to be not seen.
If you think this is still unresolved and have more information to bring, please create another issue.

This is an automated comment setup because @qdm12 is the sole maintainer of this project
which became too popular to monitor issues closed.

@virtitnerd
Copy link
Author

Update your servers using the method in the wiki is still using an outdated servers list for ProtonVPN.
Comparing that against the ProtonVPN API, there is more P2P servers listed on there than there is on the servers.json that's being pulled

@qdm12
Copy link
Owner

qdm12 commented Jul 26, 2024

After updating servers.json locally, I get 684 Protonvpn servers. The ProtonVPN API has 6064 logical servers, 6255 physical servers, and 56 disabled physical servers.
Each physical server has a single entry IP address, and thus Gluetun deduplicates physical servers by entry IP addresses, hence going from 6199 physical servers to 684 physical servers.

key := entryIP.String()
server, ok := its[key]
if ok {
return
}

For example, the entry IP address 79.135.105.24 is there multiple times since it has several exit IPs. From our client perspective, we only care about the entry IP, that is why it's deduplicated.

Are you then still missing servers? If so which ones?

@qdm12 qdm12 changed the title VPN provider update: ProtonVPN Server List Update ProtonVPN servers missing after update Jul 26, 2024
@christianmerges

This comment was marked as off-topic.

@qdm12
Copy link
Owner

qdm12 commented Aug 17, 2024

Closing this due to inactivity and this seems resolved.

@qdm12 qdm12 closed this as not planned Won't fix, can't repro, duplicate, stale Aug 17, 2024
Copy link
Contributor

Closed issues are NOT monitored, so commenting here is likely to be not seen.
If you think this is still unresolved and have more information to bring, please create another issue.

This is an automated comment setup because @qdm12 is the sole maintainer of this project
which became too popular to monitor issues closed.

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