mirror of
https://github.com/meshtastic/firmware.git
synced 2025-04-25 17:42:48 +00:00
Separate host/port before checking for private IP (#5630)
This commit is contained in:
parent
df63423cdc
commit
398d29064e
@ -227,6 +227,16 @@ bool isPrivateIpAddress(const IPAddress &ip)
|
|||||||
}
|
}
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
std::pair<std::string, uint16_t> parseHostAndPort(std::string address, uint16_t port = 0)
|
||||||
|
{
|
||||||
|
const size_t delimIndex = address.find_first_of(':');
|
||||||
|
if (delimIndex > 0) {
|
||||||
|
port = std::stoul(address.substr(delimIndex + 1, address.length()));
|
||||||
|
address.resize(delimIndex);
|
||||||
|
}
|
||||||
|
return std::make_pair(std::move(address), port);
|
||||||
|
}
|
||||||
} // namespace
|
} // namespace
|
||||||
|
|
||||||
void MQTT::mqttCallback(char *topic, byte *payload, unsigned int length)
|
void MQTT::mqttCallback(char *topic, byte *payload, unsigned int length)
|
||||||
@ -302,7 +312,8 @@ MQTT::MQTT() : concurrency::OSThread("mqtt"), mqttQueue(MAX_MQTT_QUEUE)
|
|||||||
}
|
}
|
||||||
|
|
||||||
IPAddress ip;
|
IPAddress ip;
|
||||||
isMqttServerAddressPrivate = ip.fromString(moduleConfig.mqtt.address) && isPrivateIpAddress(ip);
|
isMqttServerAddressPrivate =
|
||||||
|
ip.fromString(parseHostAndPort(moduleConfig.mqtt.address).first.c_str()) && isPrivateIpAddress(ip);
|
||||||
|
|
||||||
#if HAS_NETWORKING
|
#if HAS_NETWORKING
|
||||||
if (!moduleConfig.mqtt.proxy_to_client_enabled)
|
if (!moduleConfig.mqtt.proxy_to_client_enabled)
|
||||||
@ -418,14 +429,9 @@ void MQTT::reconnect()
|
|||||||
pubSub.setClient(mqttClient);
|
pubSub.setClient(mqttClient);
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
String server = String(serverAddr);
|
std::pair<std::string, uint16_t> hostAndPort = parseHostAndPort(serverAddr, serverPort);
|
||||||
int delimIndex = server.indexOf(':');
|
serverAddr = hostAndPort.first.c_str();
|
||||||
if (delimIndex > 0) {
|
serverPort = hostAndPort.second;
|
||||||
String port = server.substring(delimIndex + 1, server.length());
|
|
||||||
server[delimIndex] = 0;
|
|
||||||
serverPort = port.toInt();
|
|
||||||
serverAddr = server.c_str();
|
|
||||||
}
|
|
||||||
pubSub.setServer(serverAddr, serverPort);
|
pubSub.setServer(serverAddr, serverPort);
|
||||||
pubSub.setBufferSize(512);
|
pubSub.setBufferSize(512);
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user