esp8266, Arduino IDE: neuer Versuch: warum wird website sofort beendet?

[Ceos]

JETZT hast du dein client.close() allerdings auch entfernt, was zu "socketverstopfung" führen kann.

[HaWe]

ach so, ich dachte, der soll auch mit weg?
soll der auch direkt dahinter?

       while(client.connected()) {
           while(client.available()) {
              Serial.write(client.read());
           }
       } 
       delay(100);
       client.stop();

- - - Aktualisiert - - -

PS,
habe es mal so gemacht,
einziger "Schönheitsfehler" jetzt noch: ich muss immer 2x hintereinander auf den "Daten absenden"-Button klicken (oder ENTER-Taste), damit usrname+pwd übertragen werden, aber nach dem 2. mal klappt es jetzt (ohne client.stop() war es auch schon so).

auch wenn ich die while-Schleife ersetze durch

while(client.connected()) {
          delay(10);
          /* while(client.available()) {
              Serial.write(client.read());
           }*/
        }

ändert sich das nicht.
(Serial ist ja stand-allone nicht mehr verfügbar!)

[Moppi]

Ich denke:

Du müsstest Dich entscheiden, ob die Connection zum Client bestehen bleibt oder geschlossen wird.

WiFiClient client = wifiserver.available();

Jetzt kannst Du mittels client mit dem Browser kommunizieren, der eine Verbindung hergestellt hat.
Die Verbindung sollte jetzt bestehen.

Da Du das jedesmal machst, wenn ein Datenaustausch mit dem Client statt gefunden hat, müsstest Du die Verbindung beenden, s. client.stop().

Der Client, also der Browser soll aber wohl wissen müssen, was passieren wird. Dazu sendest Du einen Header:

        script += ("HTTP/1.1 200 OK \n");
        script += ("Content-Type: text/html \n");
        script += ("\n");  //  do not forget this one //????

In diesem steht aber nicht, dass die Connection geschlossen werden soll.
Vielleicht mal mit einem korrekten Header anfangen:

        script += ("HTTP/1.1 200 OK\r\n");
        script += ("Content-Type: text/html\r\n");
        script += ("Connection: close\r\n");
        ...

Von vorne:


Zuerst mit WiFiClient client = wifiserver.available(); ein Objekt holen, dass Kommunikation mit dem Browser erlaubt (Browser hat Connection hergestellt). Das kannst Du zur Not mit if(client)... überprüfen.

Dann prüfen, ob der Browser Daten gesendet hat und diese alle einlesen und verarbeiten.

Wenn fest steht, was er gesendet hat und die Daten (User und Passwort) richtig waren, dann die Seitendaten senden (inkl. dem Header wie oben).

Nach dem Datentransfer dem Browser mit delay(x) etwas Zeit geben, die Daten zu empfangen.

Dann, am Schluss, zum Schließen der Connection client.stop() verwenden.



MfG

[HaWe]

Ich denke:

Du müsstest Dich entscheiden, ob die Connection zum Client bestehen bleibt oder geschlossen wird.

WiFiClient client = wifiserver.available();

Jetzt kannst Du mittels client mit dem Browser kommunizieren, der eine Verbindung hergestellt hat.
Die Verbindung sollte jetzt bestehen.

Da Du das jedesmal machst, wenn ein Datenaustausch mit dem Client statt gefunden hat, müsstest Du die Verbindung beenden, s. client.stop().

Der Client, also der Browser soll aber wohl wissen müssen, was passieren wird. Dazu sendest Du einen Header:

        script += ("HTTP/1.1 200 OK \n");
        script += ("Content-Type: text/html \n");
        script += ("\n");  //  do not forget this one //????

In diesem steht aber nicht, dass die Connection geschlossen werden soll.
Vielleicht mal mit einem korrekten Header anfangen:

        script += ("HTTP/1.1 200 OK\r\n");
        script += ("Content-Type: text/html\r\n");
        script += ("Connection: close\r\n");
        ...

Von vorne:


Zuerst mit WiFiClient client = wifiserver.available(); ein Objekt holen, dass Kommunikation mit dem Browser erlaubt (Browser hat Connection hergestellt). Das kannst Du zur Not mit if(client)... überprüfen.

Dann prüfen, ob der Browser Daten gesendet hat und diese alle einlesen und verarbeiten.

Wenn fest steht, was er gesendet hat und die Daten (User und Passwort) richtig waren, dann die Seitendaten senden (inkl. dem Header wie oben).

Nach dem Datentransfer dem Browser mit delay(x) etwas Zeit geben, die Daten zu empfangen.

Dann, am Schluss, zum Schließen der Connection client.stop() verwenden.



MfG

hi,
WiFiClient client = wifiserver.available();
habe ich ja schon ganz oben im Code stehen.

Ich habe nun die alten 3 Codezeilen durch die 3 neuen ersetzt, allerdings baut er jetzt gar keine website mehr auf - was habe ich übersehen? (ggf: wie lautet der komplette neue Code, falls nicht nur 1 Zeile betroffen ist?)

[Moppi]

ich habs mal etwas umgestellt - versuch das mal:

bool authorized=false;


void handleNotAuthorized() {
  String readString = "";
  char   strinput[MAXLEN], strupwd[TOKLEN], struname[TOKLEN] ;
  
  WiFiClient client = wifiserver.available();


  if (client) {


          //---------------------------------------
          // debug
          // authorized=true;


          strcpy(strinput, "");
          strcpy(strupwd, "");
          strcpy(struname, "");


  


          while ( client.connected() ) {
            if (authorized) return;


            if ( client.available() ) {
              char c = client.read();


              //read char by request
              readString = "";
              while ( (readString.length() < TOKLEN) && (c != '\n') ) {
                  readString += c;
                  c = client.read();
              }
        
              readString.toCharArray(strinput, MAXLEN);
         
              if(strstr(website_upwd,strupwd)!=NULL & strstr(website_uname,struname)!=NULL)
              {
                authorized = true;
                readString = "";
                return;           
                // return;  // <<< selbes Ergebnis, egal ob kommentiert oder auskommentiert
              }    
      
              //if HTTP request has ended
              if (c == '\n') {
                 //client.flush();


                //now output html data header


                String script = "";


                script += ("HTTP/1.1 200 OK\r\n");
                script += ("Content-Type: text/html\r\n");
                script += ("Connection: close\r\n");
                script += ("\n");  //  do not forget this one //????
                script += ("<!DOCTYPE html> \n");
                script += ("<html> \n");
        
                script += ("<head> \n");


                // utf-8 für "°" Zeichen
                script +=  "<meta http-equiv=\"Content-Type\" content=\"text/html; charset=utf-8\"> \n" ;
                script +=  "<title>" ;        
                script +=  "</title> \n" ;
                script +=  "</head> \n" ;
                script +=  "<body> \n" ;
        
                script += ("<h2><p style=\"color:rgb(255,0,191);\"> log in to proceed: </p> </h2> \n");


                script += ("<FORM ACTION='/' method=GET > \n");
                script += ("<h2>user name:  <INPUT TYPE=text NAME='uname' VALUE=''  MAXLENGTH='50'> </h2> \n");
                script += ("<h2>password :  <INPUT TYPE=PASSWORD NAME='upwd' VALUE='' MAXLENGTH='50'> </h2> \n");


                script += ("<h2><INPUT TYPE=SUBMIT></h2> \n");


                script += ("</FORM> \n");
                script += ("<BR> \n");
                script += ("</body> \n");
                script += ("</html> \n");


                client.println(script);
                delay(100);
        
                //stopping client
                client.stop();
                //clearing string for next read
                //readString = "";
              }
            }
            //delay(1);
          }
  }
}

[HaWe]

leider springt er jetzt sofort wieder raus, wie ganz am Anfang,
und wenn ich einsetze vor client.stop();

          //stopping client
          while(client.connected()) {
             delay(10);             
          }
          client.stop();

dann baut er sie auf, beibt aber ewig drin hängen, auch nach korrektem login+pwd

[Moppi]

strupwd und struname werden doch überhaupt nicht aus dem gelesenen String readString extrahiert, aber anschließend verglichen und dann davon abhängig gemacht, ob authorized = true;

Das haut irgendwie hinten und vorne noch nicht hin.