-
Notifications
You must be signed in to change notification settings - Fork 18
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
Issues with DHCP entries containing hostnames with invalid characters #1
Comments
Awesome, I look forward to your pull request! I'm familiar with the problem On 12 May 2016 at 18:46, Jeroen Wiert Pluimers notifications@github.com
|
Hi Tyler, Thanks. So you still use Mikrotik devices? I ask because I want to put up a collection of RouterOS scripts online I've recently asked a few script related questions on the mikrotik forums Regards, --jeroen |
That's a kind offer, thanks! Email me at tyler@tolaris.com (tyler tolaris I do use two Mikrotik routers for some colocated servers I own, which means On 15 May 2016 at 16:04, Jeroen Wiert Pluimers notifications@github.com
|
What is best to do with unacceptable symbols? Remove or replace by hex code? |
I'm working on the latter. See https://github.com/jpluimers/mikrotik-routeros/blob/master/scripts/Function.stripInvalidHostNameCharactersFromString.rsc for my work so far. |
@mmv-ru in the link a LOG output by the code I have so far. I need to add a https://gist.github.com/jpluimers/5555abaf0d46357203638b674930abfe |
fix Issue #1 and some other improvements
Hostnames should only include a limited set of characters: https://en.wikipedia.org/wiki/Hostname#Restrictions_on_valid_host_names
I've two types of devices that fail here:
They are added with invalid hostnames into the DNS.
In due time you will get a pull request that solves this and will also remove those added invalid hostnames.
It is going to take a while as the Mikrotik RouterOS Scripting language has very limited documentation, has some issues with associative arrays and the
:pick
function, and in general has a steep learning curve.--jeroen
The text was updated successfully, but these errors were encountered: