Services API GetSetting is Case Sensitive

[drew@…]

​https://github.com/MythTV/mythtv/blob/ef8694a20a3db8f1da91e78074314cdc635f6cf9/mythtv/programs/mythbackend/services/myth.cpp#L645

GetSetting performs a SQL select against the given hostname, but also against any hostname not defined for a key. Then it loops through to find the corresponding value to use, comparing the provided hostname with the DB hostname. In the case of MySQL and other default case-insensitive databases, this causes problems.

In this case where the function is provided the hostname 'HTPC' and key of 'BackendServerIP'. The DB has hostname 'htpc' instead. The select statement returns the correct row, however the if statement does not select this. Instead it returns no value at all because it didn't 'find' it in the loop.

A simple patch:

(sHostName.toLower() == query.value(1).toString().toLower())

[Bill Meek]

Please add the output of these 3 commands:

mysql> SELECT DISTINCT hostname FROM settings;

$ hostname

$ curl yourBackendIPorHostname:6544/Myth/GetHostName

[drew@…]

mysql> SELECT DISTINCT hostname FROM settings;
+-----------------+
| hostname        |
+-----------------+
| Backend         |
| NULL            |
| HTPC1           |
| BedroomPi       |
| DeskPi          |
| LivingRoom      |
| localhost       |
| LoftRPI2        |
| xbian           |
+-----------------+
9 rows in set (0.03 sec)

$ hostname
Backend

$ curl 10.0.0.254:6544/Myth/GetHostName
<?xml version="1.0" encoding="UTF-8"?><String>Backend</String>

This doesn't show the "htpc" lower-case version because I have since removed that client and its associated settings.

[Bill Meek]

Fixed in 0.28 (which entered Beta today.) The following example now works:

backend0:6544/Myth/GetSetting?hostname=BACKEND0&KEY=backendserverport

Myth/GetHostName returns: backend0 for this example. Both the HostName and Key values are case agnostic (as are the parameter names themselves.)