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

Script error at email check for RouterOS 7.12.1 #48

Open
AndresCampus opened this issue Jan 3, 2024 · 5 comments
Open

Script error at email check for RouterOS 7.12.1 #48

AndresCampus opened this issue Jan 3, 2024 · 5 comments

Comments

@AndresCampus
Copy link

I get an error after RouterOS update in section: #Check proper email config
The expression [/tool e-mail get address] is not valid because of the "address" value
Look like is not longer valid value in that context. I changed to "user", but I am not sure if the meaning of values are similar.
Current RouterOS: 7.12.1
Thank you

@AndresCampus
Copy link
Author

AndresCampus commented Jan 3, 2024

Looks like "address" was the IP address (of server?), but I can not find "address" anymore inside /tool/e-mail. BTW, there is a "last-address" value. I don't know if they are the same thing. Maybe is better to check if "server" is set. My script copy is working again after solving the issue by changing that value with a valid one

@stefan-b-b
Copy link

Hi - The script stopped working when trying to validate the below parts. I just disabled for now to fix.
-##

#Script messages prefix
:local SMP "Bkp&Upd:"

:log info "\r\n$SMP script "Mikrotik RouterOS automatic backup & update" started.";
:log info "$SMP Script Mode: $scriptMode, forceBackup: $forceBackup";

#Check proper email config
#:if ([:len $emailAddress] = 0 or [:len [/tool e-mail get address]] = 0 or [:len [/tool e-mail get from]] = 0) do={

:log error ("$SMP Email configuration is not correct, please check Tools -> Email. Script stopped.");

:error "$SMP bye!";

#}

#Check if proper identity name is set
#if ([:len [/system identity get name]] = 0 or [/system identity get name] = "MikroTik") do={

:log warning ("$SMP Please set identity name of your device (System -> Identity), keep it short and #informative.");

#};

@beeyev
Copy link
Owner

beeyev commented Apr 1, 2024

I checked several times, everything seems to work correctly.

@crowfeatherjoe
Copy link

running on 7.16rc2 here, i had to change [tool e-mail get address] to [tool e-mail get server]

you may need to update the script to the most recent release as there is a check for this

@beeyev
Copy link
Owner

beeyev commented Aug 31, 2024

Thanks @crowfeatherjoe , I will check it!

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

4 participants