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

add security note about getAdmin #13

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

Garfonso
Copy link

No description provided.

README.md Outdated
### Security Note
Please note that the tool GetAdmin allows all users with access to the IP & Port configured (see below) vast control of your device. Please make sure that you secure access to the tool, for example using the windows firewall and allow access only from the IP of your ioBroker Server.

Because of this remote control over your device the GetAdmin tool will obviously look like a trojan horse to most virus scanners. In case of a detection you should carefully check the full results and see if there are any malcious actions detected (most virus scanners already complain about starting / stopping certain Applications / Services on your device, which the tool obviously has to do). But still bear in mind: if anybody else can access GetAdmin, he can control also start / stop Applications on your computer or even shut it down. So make sure only you / this adapter can.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe replace "most" by "some" :-)

@Apollon77
Copy link

@Mic-M Can we merge this fast in please??

@Garfonso
Copy link
Author

@Mic-M & @Apollon77
this is still open. Any second thoughts?

@Apollon77
Copy link

@Mic-M ?? Any way ? Please else condier moving the adapter into community support!

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

Successfully merging this pull request may close these issues.

2 participants