Skip to content

WifiMulti resilience updates #2775

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 13 commits into from
Closed

WifiMulti resilience updates #2775

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
f8eaa42
Improvements to WiFiMulti resilience
Daemach May 13, 2019
d5b229c
Added example file for new options
Daemach May 13, 2019
0fab2f8
Modified to include connection testing at startup
Daemach May 14, 2019
fa7d1be
Merge remote-tracking branch 'upstream/master' into wifiMultiExpand
Daemach May 14, 2019
a406a3f
Changed strictmode function, default testPhrase
Daemach May 15, 2019
f2b2ff9
Stability fixes
Daemach Jun 6, 2019
3c5ecab
Merge remote-tracking branch 'upstream/master' into wifiMultiExpand
Daemach Jun 6, 2019
15af6ef
Added upstream
Daemach Jun 6, 2019
592079e
testConnection timeout fix
Daemach Jun 7, 2019
d0d4d17
Removed vscode folders
Daemach Jun 21, 2019
838b075
More vscode cleanup
Daemach Jun 21, 2019
c587b44
Merge remote-tracking branch 'upstream/master' into wifiMultiExpand
Daemach Jul 9, 2019
19ad37e
Rebase
Daemach Jul 10, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
testConnection timeout fix 
Copy/paste error :/
  • Loading branch information
@Daemach
Daemach committed Jun 7, 2019
commit 592079ecd580301df2650164e12cb889ee8d8311
5 changes: 4 additions & 1 deletion libraries/WiFi/src/WiFiMulti.cpp
View file
Open in desktop
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please merge the master branch and resolve the conflicts ?

Original file line number Diff line number Diff line change
Expand Up @@ -289,14 +289,17 @@ bool WiFiMulti::testConnection(){
bool bSuccess = false;
timeout = millis();
while(client.available()) {
if (millis() - timeout > 5000) {
if (millis() - timeout < 5000) {
bSuccess = client.find(_testPhrase.c_str());
if (bSuccess){
log_i("Success. Found test phrase");
} else {
log_e("Failed. Can't find test phrase");
}
return bSuccess;
} else {
log_e("Test phrase check timed out!");
break;
}
}
}
Expand Down