Skip to content

Conversation

@su-haris
Copy link
Contributor

@su-haris su-haris commented Jul 8, 2023

Sometimes from ip-api, either ISP or ASN is available, but it would not show since the condition is not satisfied. Fix to show if either value is present.

Also added an Asian location (Jakarta, ID) to iperf test.

su-haris added 2 commits June 26, 2023 21:44
Sometimes from ip-api, either ISP or ASN is available, but it would not show since the condition is not satisfied. Fix to show if either value is present
@su-haris
Copy link
Contributor Author

su-haris commented Jul 8, 2023

Weird that there is some spacing issue on the PR.

image

@masonr
Copy link
Owner

masonr commented Jul 13, 2023

Convert spaces to tabs and that should fix the spacing issue.

@su-haris
Copy link
Contributor Author

Thanks for the suggestion @masonr , that fixed it!

@masonr masonr merged commit af78f35 into masonr:master Sep 3, 2023
@masonr
Copy link
Owner

masonr commented Sep 6, 2023

This PR was merged a few days ago, but I have just removed the Biznet (Jakarta, ID) iperf3 server from the list. I've had this particular one added before and it was never very stable. After noticing many failures on that location the since the merge, I decided to pull it again. Some of the ports weren't responding, IPv6 wasn't working to that location, uptime isn't that great -- will be looking for another location in the region use instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants