mirror of
https://github.com/alexhopeoconnor/WiFiManager.git
synced 2026-10-04 02:48:13 +10:00
Redact WiFiManager diagnostic values
This commit is contained in:
@@ -1,5 +1,9 @@
|
||||
# Changelog
|
||||
|
||||
## 3.0.1
|
||||
|
||||
- Redact WiFi, AP, portal-form, and custom-parameter values from diagnostic logs.
|
||||
|
||||
## 3.0.0
|
||||
|
||||
- Establish `device-framework` as the independently maintained canonical branch.
|
||||
|
||||
@@ -986,7 +986,7 @@ bool WiFiManager::wifiConnectNew(String ssid, String pass,bool connect){
|
||||
bool ret = false;
|
||||
#ifndef WM_NO_LOG
|
||||
log(WiFiManagerLogLevel::Info, kWiFiMgrLogSubsystem, F("Connecting to NEW AP:"),ssid);
|
||||
log(WiFiManagerLogLevel::Trace, kWiFiMgrLogSubsystem,F("Using Password:"),pass);
|
||||
log(WiFiManagerLogLevel::Trace, kWiFiMgrLogSubsystem, F("WiFi password supplied: [redacted]"));
|
||||
#endif
|
||||
WiFi_enableSTA(true,storeSTAmode); // storeSTAmode will also toggle STA on in default opmode (persistent) if true (default)
|
||||
WiFi.persistent(true);
|
||||
@@ -1007,7 +1007,7 @@ bool WiFiManager::wifiConnectDefault(){
|
||||
|
||||
#ifndef WM_NO_LOG
|
||||
log(WiFiManagerLogLevel::Info, kWiFiMgrLogSubsystem, F("Connecting to SAVED AP:"),WiFi_SSID(true));
|
||||
log(WiFiManagerLogLevel::Trace, kWiFiMgrLogSubsystem,F("Using Password:"),WiFi_psk(true));
|
||||
log(WiFiManagerLogLevel::Trace, kWiFiMgrLogSubsystem, F("Using saved WiFi credentials"));
|
||||
#endif
|
||||
|
||||
ret = WiFi_enableSTA(true,storeSTAmode);
|
||||
@@ -2443,7 +2443,7 @@ void WiFiManager::debugSoftAPConfig(){
|
||||
log(WiFiManagerLogLevel::Info, kWiFiMgrLogSubsystem, F("SoftAP Configuration"));
|
||||
log(WiFiManagerLogLevel::Info, kWiFiMgrLogSubsystem, F("--------------------"));
|
||||
log(WiFiManagerLogLevel::Info, kWiFiMgrLogSubsystem, F("ssid: "),(char *) config.ssid);
|
||||
log(WiFiManagerLogLevel::Info, kWiFiMgrLogSubsystem, F("password: "),(char *) config.password);
|
||||
log(WiFiManagerLogLevel::Info, kWiFiMgrLogSubsystem, F("password: [redacted]"));
|
||||
log(WiFiManagerLogLevel::Info, kWiFiMgrLogSubsystem, F("ssid_len: "),config.ssid_len);
|
||||
log(WiFiManagerLogLevel::Info, kWiFiMgrLogSubsystem, F("channel: "),config.channel);
|
||||
log(WiFiManagerLogLevel::Info, kWiFiMgrLogSubsystem, F("authmode: "),config.authmode);
|
||||
@@ -2518,7 +2518,7 @@ boolean WiFiManager::validApPassword(){
|
||||
}
|
||||
#ifndef WM_NO_LOG
|
||||
log(WiFiManagerLogLevel::Debug, kWiFiMgrLogSubsystem,F("AccessPoint set password is VALID"));
|
||||
log(WiFiManagerLogLevel::Trace, kWiFiMgrLogSubsystem,"ap pass",_apPassword);
|
||||
log(WiFiManagerLogLevel::Trace, kWiFiMgrLogSubsystem, F("Access point password: [redacted]"));
|
||||
#endif
|
||||
}
|
||||
return true;
|
||||
|
||||
@@ -739,7 +739,7 @@ void WiFiManagerHandlers::applyWifiAndParamsFromRequest(AsyncWebServerRequest *r
|
||||
requestinfo += "\n";
|
||||
for (size_t i = 0; i < request->params(); i++) {
|
||||
const AsyncWebParameter* p = request->getParam(i);
|
||||
requestinfo += " " + p->name() + ": " + p->value() + "\n";
|
||||
requestinfo += " " + p->name() + ": [redacted, " + String(p->value().length()) + " chars]\n";
|
||||
}
|
||||
|
||||
_wm->log(WiFiManagerLogLevel::Trace, kWiFiMgrLogSubsystem, requestinfo);
|
||||
@@ -823,7 +823,8 @@ void WiFiManagerHandlers::doParamSave(WiFiManager::WiFiManagerRequestArgs reques
|
||||
|
||||
_wm->_params[i]->setValue(value.c_str(), value.length());
|
||||
#ifndef WM_NO_LOG
|
||||
_wm->log(WiFiManagerLogLevel::Debug, kWiFiMgrLogSubsystem,(String)_wm->_params[i]->getID() + ":",value);
|
||||
_wm->log(WiFiManagerLogLevel::Debug, kWiFiMgrLogSubsystem,
|
||||
(String)_wm->_params[i]->getID() + F(": [redacted, ") + String(value.length()) + F(" chars]"));
|
||||
#endif
|
||||
}
|
||||
#ifndef WM_NO_LOG
|
||||
|
||||
+1
-1
@@ -1,6 +1,6 @@
|
||||
{
|
||||
"name": "WiFiManager",
|
||||
"version": "3.0.0",
|
||||
"version": "3.0.1",
|
||||
"keywords": [
|
||||
"wifi",
|
||||
"wi-fi",
|
||||
|
||||
Reference in New Issue
Block a user