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

$ExternalIP randomly failing in Generic #944

Closed
3 tasks done
Greelan opened this issue Nov 3, 2023 · 2 comments
Closed
3 tasks done

$ExternalIP randomly failing in Generic #944

Greelan opened this issue Nov 3, 2023 · 2 comments

Comments

@Greelan
Copy link
Contributor

Greelan commented Nov 3, 2023

Bug Report

System Information

  • Windows or Linux
  • 2.4.6.6
  • Mainline

I confirm:

  • that I have searched for an existing bug report for this issue.
  • that I am using the latest available version of AMP.
  • that my operating system is up-to-date.

Symptoms

A number of Generic templates use $ExternalIP to populate the public IP.

This seems to randomly fail. On some occasions when a server is started, the IP will populate correctly; on others, it does not.

It's almost like if a fetch fails for some reason, any existing data is overwritten with an empty value, and there is no error handling to ensure a retry until good data is obtained.

Can this please be looked at?

Reproduction

See above.

@PhonicUK
Copy link
Contributor

PhonicUK commented Jan 3, 2024

I've changed the logic for this so it'll try multiple providers, and it'll try multiple times before giving up. There's no 'existing' data per-se, AMP works it out once when it starts up.

@PhonicUK
Copy link
Contributor

Fixed in 2.5

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

No branches or pull requests

2 participants