Domoticz

Slayer007 wrote:I've read that there's no security in the bot. So anyone that knows the name of my bot, can turn my lights on, for example.
I read in this forum that you can make a user variabele in domoticz. But i can't get it to work. I don't know exactly what to do. Can anyone help me or is there another security that can be implemented.

WLName = 'TelegramBotWhiteListedIDs'

1234343423|0982384343

MacJL wrote:Hello,

I have this error when trying to obtain temperature :

Code: Select all

/usr/bin/lua5.2: /home/pi/dtgbot/lua//temperature.lua:31: attempt to concatenate global 'Humidity' (a nil value)
stack traceback:
        /home/pi/dtgbot/lua//temperature.lua:31: in function 'temperature'
        /home/pi/dtgbot/lua//temperature.lua:44: in function 'handler'
        /home/pi/dtgbot/dtgbot.lua:293: in function 'HandleCommand'
        /home/pi/dtgbot/dtgbot.lua:400: in function 'on_msg_receive'
        /home/pi/dtgbot/dtgbot.lua:490: in main chunk
        [C]: in ?

function get_temperature(DeviceName)
  idx = idx_from_name(DeviceName,'devices')
  if idx == nil then
    return DeviceName, -999, -999, 0
  end
-- Determine temperature
  t = server_url.."/json.htm?type=devices&rid=" .. idx
  print ("JSON request <"..t..">");
  jresponse, status = http.request(t)
  decoded_response = JSON:decode(jresponse)
  result = decoded_response["result"]
  record = result[1]
  Temperature = record["Temp"]
  if record["Type"] == "Temp + Humidity" then
    Humidity = record["Humidity"]
  else
    Humidity = -999
  end
  LastUpdate = record["LastUpdate"]
  DeviceName = record["Name"]
  return DeviceName, Temperature, Humidity, LastUpdate;
end

function temperature(DeviceName)
  local response = ""
  DeviceName, Temperature, Humidity, LastUpdate = get_temperature(DeviceName)
  if Temperature == -999 then
    print(DeviceName..' does not exist')
    return 1, DeviceName..' does not exist'
  end
  if Humidity ~= -999 then
    print(DeviceName .. ' temperature is ' .. Temperature .. '°C and relative humidity is ' .. Humidity .. '%')
    response = DeviceName.. ' ' .. Temperature .. '°C & ' .. Humidity .. '%'
  else
    print(DeviceName .. ' temperature is ' .. Temperature .. '°C')
    response = DeviceName.. ' ' .. Temperature .. '°C'
  end
  return status, response;
end

MacJL wrote:I have created a simple script that add the chat ids to telegram TelegramBotWhiteListedIDs user variable.
The steps :

Spoiler: show

# Stop dtgbot
sudo service dtgbot stop
sleep 60
# Send a telegram message to the bot
# Execute the script
chmod +x authorize.bash
./authorize.bash
# Restart dtgbot
sudo service dtgbot start

If you want to add the script to the git repository and the wiki page, you can do it.

The content of authorize.bash

Code: Select all

#!/bin/bash

# Getting Domoticz user variables
JMESSAGES=$( curl -sS "https://api.telegram.org/bot$TelegramBotToken/getUpdates" )

# Getting Telegram Bot last messages
DOMOVAR=$( curl -sS "http://$DomoticzIP:$DomoticzPort/json.htm?type=command&param=getuservariables")

# Parsing User variable "TelegramBotWhiteListedIDs" Value and IDX
DOMOVAR_VAL=$( echo $DOMOVAR | jq -c '.result[]|select(.Name == "TelegramBotWhiteListedIDs").Value' | sed "s/\"//g" )
DOMOVAR_IDX=$( echo $DOMOVAR | jq -c '.result[]|select(.Name == "TelegramBotWhiteListedIDs").idx'   | sed "s/\"//g" )

# For each user or group chat
echo $JMESSAGES | jq -c .result[].message.chat | sort -u | while read line
do
        FIRSTNAME=$(  echo $line | jq -c .first_name | sed "s/\"//g" )
        LASTNAME=$(   echo $line | jq -c .last_name  | sed "s/\"//g" )
        USERNAME=$(   echo $line | jq -c .username   | sed "s/\"//g" )
        GROUPTITLE=$( echo $line | jq -c .title      | sed "s/\"//g" )
        CHATID=$(     echo $line | jq -c .id         | sed "s/\"//g" )

        # Gessing if it's a user or a group chat
        if [[ $GROUPTITLE == "null" ]]
        then
                DESTNAME="user $FIRSTNAME $LASTNAME, $USERNAME with chat id : \"$CHATID\""
        else
                DESTNAME="group $GROUPTITLE with chat id : \"$CHATID\""
        fi

        # Is the CHATID already authorized?
        if [ $( echo "|$DOMOVAR_VAL|" | sed "s/\"/\|/g" | grep "|$CHATID|" | wc -l ) -eq 1 ]
        then
                echo "The $DESTNAME is already authorized"
        else
                echo "Adding $DESTNAME to $DOMOVAR_VAL"
                DOMOVAR_VAL="$DOMOVAR_VAL|$CHATID"
                curl -sS "http://$DomoticzIP:$DomoticzPort/json.htm?type=command&param=updateuservariable&idx=$DOMOVAR_IDX&vname=TelegramBotWhiteListedIDs&vtype=2&vvalue=$DOMOVAR_VAL" | jq -c .status
        fi
done

bizziebis wrote:Is dtgbot Raspberry only? I followed your steps but get :

Spoiler: show

Code: Select all

//usr/bin/lua5.2: error loading module 'socket.core' from file '/usr/local/lib/lua/5.2/socket/core.so':
        /usr/local/lib/lua/5.2/socket/core.so: cannot open shared object file: No such file or directory
stack traceback:
        [C]: in ?
        [C]: in function 'require'
        /usr/local/share/lua/5.2/socket.lua:12: in main chunk
        [C]: in function 'require'
        /usr/local/share/lua/5.2/socket/http.lua:10: in main chunk
        [C]: in function 'require'
        /home/pi/dtgbot/dtgbot.lua:65: in main chunk
        [C]: in ?

Edit: Nevermind, I see I have to find them myself and compile. Maybe next time

simonrg wrote: All my temperature sensors return Humidity as well, so it always works for me, , obviously not all temperature sensors do, the code needs to check the type of the sensor and then only return humidity from ones that do. If you replace these routines in temperature.lua with the code below, then it should work ok:

Spoiler: show

function get_temperature(DeviceName)
idx = idx_from_name(DeviceName,'devices')
if idx == nil then
return DeviceName, -999, -999, 0
end
-- Determine temperature
t = server_url.."/json.htm?type=devices&rid=" .. idx
print ("JSON request <"..t..">");
jresponse, status = http.request(t)
decoded_response = JSON:decode(jresponse)
result = decoded_response["result"]
record = result[1]
Temperature = record["Temp"]
if record["Type"] == "Temp + Humidity" then
Humidity = record["Humidity"]
else
Humidity = -999
end
LastUpdate = record["LastUpdate"]
DeviceName = record["Name"]
return DeviceName, Temperature, Humidity, LastUpdate;
end

function temperature(DeviceName)
local response = ""
DeviceName, Temperature, Humidity, LastUpdate = get_temperature(DeviceName)
if Temperature == -999 then
print(DeviceName..' does not exist')
return 1, DeviceName..' does not exist'
end
if Humidity ~= -999 then
print(DeviceName .. ' temperature is ' .. Temperature .. '°C and relative humidity is ' .. Humidity .. '%')
response = DeviceName.. ' ' .. Temperature .. '°C & ' .. Humidity .. '%'
else
print(DeviceName .. ' temperature is ' .. Temperature .. '°C')
response = DeviceName.. ' ' .. Temperature .. '°C'
end
return status, response;
end

I have updated the master branch in github with these corrections, thanks.

MacJL wrote:Still the same issue with this update. In fact, this sensor is a not a temperature only sensor, but a temperature + Humidity + barometer sensor. If you need more logs, I can provides them.

Spoiler: show

Code: Select all

function get_temperature(DeviceName)
  idx = idx_from_name(DeviceName,'devices')
  if idx == nil then
    return DeviceName, -999, -999, -999, 0
  end
  Temperature = -999
  Humidity = -999
  Pressure = -999
-- Determine temperature
  t = server_url.."/json.htm?type=devices&rid=" .. idx
  print ("JSON request <"..t..">");
  jresponse, status = http.request(t)
  decoded_response = JSON:decode(jresponse)
  result = decoded_response["result"]
  record = result[1]
  DeviceType = record["Type"]
  if DeviceType == "Temp" then
    Temperature = record["Temp"]
  else
    if DeviceType == "Humidity" then
      Humidity = record["Humidity"]
    else
      if DeviceType == "Temp + Humidity" then
        Temperature = record["Temp"]
        Humidity = record["Humidity"]
      else
        if DeviceType == "Temp + Humidity + Baro" then
          Temperature = record["Temp"]
          Humidity = record["Humidity"]
          Pressure = record["Barometer"]
        end
      end
    end
  end
  LastUpdate = record["LastUpdate"]
  DeviceName = record["Name"]
  return DeviceName, Temperature, Humidity, Pressure, LastUpdate;
end

function temperature(DeviceName)
  local response = ""
  DeviceName, Temperature, Humidity, Pressure, LastUpdate = get_temperature(DeviceName)
  if Temperature == -999 and Humidity == -999 and Pressure == -999 then
    print(DeviceName..' does not exist')
    return 1, DeviceName..' does not exist'
  else
    if Temperature == -999 and Pressure == -999 then
      print(DeviceName .. ' relative humidity is ' .. Humidity .. '%')
      response = DeviceName.. ' ' .. Humidity .. '%'
    else  
      if Pressure ~= -999 then
        print(DeviceName .. ' temperature is ' .. Temperature .. '°C, relative humidity is ' .. Humidity .. '% and pressure is '.. Pressure..'hPa')
        response = DeviceName.. ' ' .. Temperature .. '°C & ' .. Humidity .. '% & '.. Pressure .. 'hPa'
      else
        if Humidity ~= -999 then
          print(DeviceName .. ' temperature is ' .. Temperature .. '°C and relative humidity is ' .. Humidity .. '%')
          response = DeviceName.. ' ' .. Temperature .. '°C & ' .. Humidity .. '%'
        else
          print(DeviceName .. ' temperature is ' .. Temperature .. '°C')
          response = DeviceName.. ' ' .. Temperature .. '°C'
        end
      end
    end
  end
  return status, response;
end

MacJL wrote:Thanks!

Ther is no mare crashes, but it reply 'Salon does not exist', even If ther is a Temp Sensor that is called 'Salon'.It seems to be the fact that I have two devices which are called Salon : A temperature Sensor, and a switch :

Spoiler: show

Code: Select all

.{"ok":true,"result":[]}
.{"ok":true,"result":[{"update_id":726956523,
"message":{"message_id":249,"from":{"id":34862038,"first_name":"Jean-Laurent","last_name":"Girod","username":"macjl"},"chat":{"id":34862038,"first_name":"Jean-Laurent","last_name":"Girod","username":"macjl"},"date":1440071800,"text":"\/temperature Salon"}}]}
Message: 1
update_id       726956523
/temperature Salon
No on WhiteList: 2
WhiteList: xxxxxxxxxx
Handle command function started with /temperature Salon and xxxxxxxxxx
JSON request <http://127.0.0.1:8080/json.htm?type=devices&order=name>
25
58
JSON request <http://127.0.0.1:8080/json.htm?type=devices&rid=58>
Salon does not exist
https://api.telegram.org/botxxxxxx:xxxxxx/sendMessage?timeout=60&chat_id=xxxxxx&reply_to_message_id=249&text=Salon+does+not+exist
200
Succesfully handled incoming request
TelegramBotOffset 726956524
JSON request <http://127.0.0.1:8080/json.htm?type=command&param=updateuservariable&idx=2&vname=TelegramBotOffset&vtype=integer&vvalue=726956524>

Maybe you should send a reply for each device with the same name, instead of just the last one?

Slayer007 wrote:Yep! And tried to remove everything and start over, but that also doesn't work.

Edit1:

I think it does have something to do with authorization. Check both images. One of them is mine and one of them are my parents. I don't know in which authorization I have to look.

Edit2: I think it works now. The uservariable in domoticz was set on 1. When I changed it to 2 it started. After that I've got a lot of messages till it crashed again. I requested the latest integer uservariable and changed it 1 higher. After that it's still working. Hope for a long time!

Thnx for thinking. Maybe it was helpful to other people.