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

Add config variable "override-site-info-host" #71

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

mikenye
Copy link

@mikenye mikenye commented Mar 22, 2021

As-per #70, this change implements config variable "override-site-info-host".

Users can use piaware-config override-site-info-host x.x.x.x to override the local IP address sent to FlightAware.

I'm running this on my station, site 78418. It seems to work fine.

  • "Site Local IP" matches the IP I set override-site-info-host to
  • The web interface link points to the correct IP

I haven't figured out how to change the port yet...

@mutability
Copy link
Contributor

Port changes (or more generally, the structure of the URI shown) are not going to be do-able without somewhat invasive server-side changes (we don't store a URI server-side currently, just the local IP). Is just overriding the IP sufficient for your usecase?

@mikenye
Copy link
Author

mikenye commented Mar 22, 2021

Yes it is. I can update the docker image documentation to recommend mapping port 8080 if users want the web interface link to function as expected. Thanks.

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