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

sensitive information in ROS v7 #21

Closed
robertogaxiola opened this issue Dec 23, 2021 · 5 comments
Closed

sensitive information in ROS v7 #21

robertogaxiola opened this issue Dec 23, 2021 · 5 comments

Comments

@robertogaxiola
Copy link

There is still an issue with the hide-sensitive option, now in 7.1 the export always hide sensitive information by default, you have to use show-sensitive parameter to export those options too, i have manually changed my script in the export section.

@beeyev
Copy link
Owner

beeyev commented Jan 15, 2022

Thank you for pointing it out, the issue has been fixed!

# since RouterOS v7 it needs to be set precise that we want to export sensitive data

@beeyev beeyev closed this as completed Jan 15, 2022
@MrVodinn
Copy link

Hello @beeyev! Thank you very much for your script, it is very helpful.

Can you please double check store sensitive information? I have updated the script and use "
:local sensetiveDataInConfig true;", but passwords still not included in exported config.

Current RouterOS: 7.1.1 (stable) Dec/21/2021 11:53:05
Current routerboard FW: 7.1.1

P.S. It will be nice to have an information about version of the script in the notification mail.

@beeyev
Copy link
Owner

beeyev commented Jan 17, 2022

Can you please double check store sensitive information? I have updated the script and use " :local sensetiveDataInConfig true;", but passwords still not included in exported config.

Well, that is strange, it should have worked or failed, lol. I will check it.

P.S. It will be nice to have an information about version of the script in the notification mail.

Great idea! Will implement that!

@beeyev beeyev reopened this Jan 17, 2022
beeyev added a commit that referenced this issue Jan 17, 2022
beeyev added a commit that referenced this issue Jan 17, 2022
beeyev added a commit that referenced this issue Jan 17, 2022
@beeyev
Copy link
Owner

beeyev commented Jan 17, 2022

@MrVodinn
The issue was fixed, and your idea about the script version inside the email was implemented, thank you for collaboration!

@beeyev beeyev closed this as completed Jan 17, 2022
@MrVodinn
Copy link

@beeyev
I have checked - everything works great.
Passwords are saved in the configuration export file, and recovery from the export file went without any errors on RouterOS 7.1.1.
I also want to note that the value of the field "sensetiveDataInConfig" became true by default, this is a very good solution, especially for beginners.

Thanks a lot again!

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

3 participants