-
Notifications
You must be signed in to change notification settings - Fork 40
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
Improve XLX override #15
Conversation
Existing XLX override code would only allow to override default "room", this patch improves it to allow to override XLX ip (if specified*) or to add a completely new XLX host. * i.e. to override just the default "room", user should add on the override file something like: 012;;4022 or to override ip also: 012;194.38.140.205;4022
Not so sure about this one, do we actually want the IP's to be over-ridden, since these are updated daily as part of the update process? If I am missing the point here, please tell me :) |
Well, all lists (DPlus, DExtra, DCS, ...) are updated daily... but in some cases we may want to override them for some reason... for example with the 012 in the example above, it's a real case for me, actually I override it to the Portuguese DCS012 server ip (now running a XLX daemon), because the official XLX012 in the list is another one... I guess there are many of these cases, as you surely know at least lower DCS and XRF numbering doesn't match, then when they migrate to XLX only one will be the "official" XLX one, and user may want to manually override it, you see what I mean... Btw, as I did mention above, my code will still allow the current behavior of overriding just "room" if user doesn't specify the ip on the override file - what IMO makes sense anyway - why user will specify an ip on the override file that will be ignored? :) |
Can do the same editing XLXHost.txt and add the changes on /root, this will override the system XLXHosts, but ok. |
Jorge: Yes, the idea is exactly to allow user to be able to override it at the /root/XLXHost.txt file - currently you can't do it - as current code only allows to override default "room" and will ignore the ip you set on that override file :) And concerning your question, obviously users that choose to override it will not be able to connect to the official XLX... This patch is just intended to allow XLX override mechanism (actually handled by HostFilesUpdate.sh) to behave similarly to other override files (these loaded and handled by the gateway apps, unlike XLX one). |
The XLX list comes from the http://xlxapi.rlx.lu/api.php?do=GetXLXDMRMaster list (directly from the XLX registry) and is NOT based on the DCS/XRF/REF lists. So when you use an XLX with DMR - the number/IP should always be correct (against the XLX master list). Why would your XLX show up with a different number? |
narspt: Yes, you're rigth!! my solution is only available for "new" systems, don´t override the old ones. |
Andy: The problem comes when the XLX born! as multiprotocol system should never have been "allowed" to register numbers below 32, there were before 32 DCS's, now problem arises, the DCS012 was transformed into XLX012 and someone occupied that position. We did our best, stating that only DCS mode can be used. |
If you need to be able to add hosts to the XLX list - we can certainly add code to make that simple, since I do believe we should be supporting that. |
I understand the XLX/DCS/REF/XRF overlap issue - we maintain our own host files for that reason, I've given up trusting any external source |
I understand it. But as Jorge said, imagine (just an example) that XRF012 owners migrate to XLX daemon, then DCS012 owners also migrate to XLX daemon, etc... (ok, main purpose of each will be to serve different protocols, but...) actually all of them will be XLX012 servers... however only one of them will be "the official XLX012" listed at XLX registry, right? Why shouldn't advanced user be able to override it, on it's override file, to be able to use an "unofficial" one instead? Why all override files at /root are intended to allow to override ip's except the XLX one that ignores them?
The patch also allows adding new hosts from the override file ;) Do you see any specific issue/disadvantage with the extended functionality added by the patch? OFF-TOPIC: Concerning the official XLX012... I did mention it just as an example, but talking about it... actually it seems taken but not even used... look at it's dashboard http://52.26.161.5/... also comparing ip's with REF012/XRF012/DCS012 none of these seems related to XLX012? PAPA System != Western ARC? |
Andrew, I know that is not your business, but for you know DCS012 start operation on 2012/04/30 with two dns's, Torsten dns and our personnal (dcs012.xreflector.net and dstarportugal.com), at that time the XLX not exists, with the end of DCS's we assume the same DCS012 because have 30~40 users daily connected. As I told before, the problem was created by having allowed (Luc) the creation of XLX with the same number as the DCS's. |
This would be a highly useful feature for me. I want to override the IP of an XLX that I operate so that I can access it via private addressing behind a firewall. |
I see now that IP override is already implemented on 37ecf79, and overriding just room doesn't make sense anymore, closing... |
Existing XLX override code would only allow to override default "room", this patch improves it to allow to override XLX ip (if specified*) or to add a completely new XLX host.
* i.e. to override just the default "room", user should add on the override file something like:
012;;4022
or to override ip also:
012;194.38.140.205;4022