Skip to content

set max_connection from code (when less than 4 is needed) #2326

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.

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

Merged
merged 5 commits into from
Aug 1, 2016
Merged

set max_connection from code (when less than 4 is needed) #2326

merged 5 commits into from
Aug 1, 2016

Conversation

Stefano-Orsolini
Copy link
Contributor

@Stefano-Orsolini Stefano-Orsolini commented Jul 25, 2016

not very common but some might require to set the maximum number of clients, from code, to less than 4.
in my case i must allow only one client at a time.

It is not very common but some might require to set the maximum number of clients, from code, to smaller than 4.
in my case i must allow only one client at a time (TESTED WORKING)
as discussed in the .cpp (set max connections)
@Stefano-Orsolini Stefano-Orsolini changed the title Stefano orsolini maxconn 1 set max_connection from code (when less than 4 is needed) Jul 25, 2016
@codecov-io
Copy link

codecov-io commented Jul 25, 2016

Current coverage is 27.62% (diff: 100%)

Merging #2326 into master will not change coverage

@@             master      #2326   diff @@
==========================================
  Files            20         20          
  Lines          3656       3656          
  Methods         335        335          
  Messages          0          0          
  Branches        678        678          
==========================================
  Hits           1010       1010          
  Misses         2468       2468          
  Partials        178        178          

Powered by Codecov. Last update b7c7bc0...6d43c2f

@@ -66,6 +66,9 @@ static bool softap_config_equal(const softap_config& lhs, const softap_config& r
if(lhs.ssid_hidden != rhs.ssid_hidden) {
return false;
}
if(lhs.max_connection != rhs.max_connection) {
return false;
Copy link
Member

Choose a reason for hiding this comment

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

Please check indentation of this line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

corrected

@igrr igrr merged commit c52088c into esp8266:master Aug 1, 2016
@Stefano-Orsolini Stefano-Orsolini deleted the Stefano-Orsolini-maxconn-1 branch August 1, 2016 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants