Page 1 of 1
Fixed: Option to listen to localhost only
Posted: Wed May 12, 2021 10:30 pm
by Mebus
Hey Teddy,
I don't want Railcontrol to be availible in my network to anybody. Is there a configuration option to make Railcontrol listen on localhost only? If not, would it be possible to add it?
Thanks
Mebus
Re: Option to listen to localhost only
Posted: Thu May 13, 2021 4:59 am
by teddy
Hi Mebus
Unfortunately there is no such option. But it is a very interesting question and it definitely makes sense to implement such an option.
I moved the topic to "Wishes and Ideas" to not forget to implement it.
Regards
Teddy
Re: Option to listen to localhost only
Posted: Thu May 13, 2021 8:46 am
by Mebus
Hi Teddy,
great!. While your already working on this topic, maybe HTTP Basic Auth would also be nice to have for the Web-UI.

If that's not possible - if you at least implement the localhost thing - I could put an nginx proxy in front of it
Mebus
Re: Option to listen to localhost only
Posted: Sun May 16, 2021 10:28 pm
by Mebus
Hallo Teddy,
I went through the source a little bit and added a configuration option to make it either listen to all addresses (::) or to localhost only (::1)

!
https://github.com/teddych/railcontrol/pull/1
I have a little problem with the "LoggingServer". It is used before the configuration has been read

How can we work around this

What is the LoggingServer used for?
At the moment it can only be restricted to localhost. Might be nice, if we could parse other IPv4 and IPv6 addresses there, too

.
Thanks
Greetings
Mebus
Re: Option to listen to localhost only
Posted: Mon May 17, 2021 6:28 am
by teddy
Hi Mebus
Thanks a lot for your contribution!
I have added some comments to the merge request. Only minor issues, but I would like to have them fixed before I merge it to the source code.
Tanks
Teddy
Re: Option to listen to localhost only
Posted: Mon May 17, 2021 9:14 pm
by Mebus
Hey Teddy,
yes, that's no problem. I will probably look into it on the weekend.
Greetings
Mebus
Re: Option to listen to localhost only
Posted: Sat May 22, 2021 7:19 pm
by teddy
Hi Mebus
Thank you very much vor your contribution!
I have merged your contribution to the master. It is visible now on GitHub.
In the official Version 20 it is still missing, because I already started to build the official release. But I think for you this is not a problem you can build it on your own.
Thanks again
Teddy