MM-Remote Android App

A Former User

My alpha user: Yeah good, i can control my MM with my phone !
My developer user : Yes… But well done… Main code of remote control have open the port mirror to everyone ! without user and pass… so every one can see your mirror !
99% of user don’t change port (lazy to read ReadMe) so th default port is 8080 ! yeah
My security point of view…: try to recode MMM-Remote-control with more security…

Sorry, it’s not for blame for program, but it’s the reality ;)

sdetweil

@Bugsounet well you would have to be on their network, so not likely exposure for most users

A Former User

@sdetweil said in MM-Remote Android App:

well you would have to be on their network, so not likely exposure for most users

sam, look any firewall scan by external source hacking … 80 8080 25 etc…

sdetweil

@Bugsounet yeh, most routers come with all ports closed. all mine in the last 10 years.

I only open ports when I need to

A Former User

I agree sam but you know most user (like me) … MM listen 8080 on internet expose …
if ip found … it’s the open for Remote control :)
browser command or androidapp -> no user /pass required

what 's why i warn :)

videogame95

I have not setup your app yet , every time I add the remote file to my config in stop what am I doing wrong

{
module: ‘MMM-Remote-Control’
// uncomment the following line to show the URL of the remote control on the mirror
// , position: ‘bottom_left’
// you can hide this module afterwards from the remote control itself
config: {
customCommand: {}, // Optional, See “Using Custom Commands” below
customMenu: “custom_menu.json”, // Optional, See “Custom Menu Items” below
showModuleApiMenu: true, // Optional, Enable the Module Controls menu
apiKey: “”, // Optional, See API/README.md for details
}
},

I set this up on my old mirror and it worked please change the code so it will work.
or am I just pasting it in wrong?

sdetweil

@videogame95 said in MM-Remote Android App:

module: ‘MMM-Remote-Control’
// uncomment the following line to show the URL of the remote control on the mirror
// , position: ‘bottom_left’
// you can hide this module afterwards from the remote control itself
config: {

rules…

the thing to the left of : does NOT need to be quoted

if the next line starts with a word (config, foo, address), then THIS line needs a trailing , at the end ( comments // on a line mean the , has to be IN FRONT of the //

the thing AFTER the : if a number or true, does NOT need to be quoted
otherwise if it is a word, or contains letters, it MUST be quoted ‘fred’, ‘0912drb’ single or double doesn’t matter as long as they are the same

so, given that, you should be able to fix this module definition…

Klettner

@videogame95 in addition to the explanations of @sdetweil you can also start your MagicMirror in the console to get hints at problems in your config.js.

First to go to the MagicMirror folder type cd ~/MagicMirror in your terminal.
Then npm start to start the mirror.

Now you should see some log messages in the terminal. If there is an error in your config.js there should also be an error message in between the logs. This message will tell you, around which line in the config.js the error occurs.
Something like:

WARNING! Could not validate config file. Starting with default configuration. Please correct syntax errors at or above this line: /home/pi/MagicMirror/config/config.js:361

In the config you posted, you are missing for example a , after ‘MMM-Remote-Control’

johnsmith332

@Klettner I will give a try hope so 🤞🤞