WebServer::args() ignores empty parameters

[tsctrl]

Board

ESP32 Dev Module

Device Description

DevKitC

Hardware Configuration

Version

v2.0.3

IDE Name

IDF Component, Arduino IDE

Operating System

Windows 10

Flash frequency

40MHz

PSRAM enabled

no

Upload speed

115200

Description

WebServer args() return wrong count if params is without equal= value

to reproduce use uri/?query as url to handle

refer example sketch as below. is this is valid result? opposite with
ESPAsyncWebServer libs request.params() count without equal= after ? as +1

Sketch

/*
    This sketch shows how to configure different external or internal clock sources for the Ethernet PHY
*/

#include <ETH.h>
#include <WiFi.h>
#include <WebServer.h>

/* 
   * ETH_CLOCK_GPIO0_IN   - default: external clock from crystal oscillator
   * ETH_CLOCK_GPIO0_OUT  - 50MHz clock from internal APLL output on GPIO0 - possibly an LAN8720 is needed for LAN8720
   * ETH_CLOCK_GPIO16_OUT - 50MHz clock from internal APLL output on GPIO16 - possibly an inverter is needed for LAN8720
   * ETH_CLOCK_GPIO17_OUT - 50MHz clock from internal APLL inverted output on GPIO17 - tested with LAN8720
*/
#define ETH_CLK_MODE    ETH_CLOCK_GPIO17_OUT

// Pin# of the enable signal for the external crystal oscillator (-1 to disable for internal APLL source)
#define ETH_POWER_PIN   -1

// Type of the Ethernet PHY (LAN8720 or TLK110)
#define ETH_TYPE        ETH_PHY_LAN8720

// I²C-address of Ethernet PHY (0 or 1 for LAN8720, 31 for TLK110)
#define ETH_ADDR        1

// Pin# of the I²C clock signal for the Ethernet PHY
#define ETH_MDC_PIN     23

// Pin# of the I²C IO signal for the Ethernet PHY
#define ETH_MDIO_PIN    18

WebServer server(80); 
static bool eth_connected = false;

void handle(){
  Serial.println(server.args());
}

void WiFiEvent(WiFiEvent_t event) {
  switch (event) {
    case SYSTEM_EVENT_ETH_START:
      Serial.println("ETH Started");
      //set eth hostname here
      ETH.setHostname("esp32-ethernet");
      break;
    case SYSTEM_EVENT_ETH_CONNECTED:
      Serial.println("ETH Connected");
      break;
    case SYSTEM_EVENT_ETH_GOT_IP:
      Serial.print("ETH MAC: ");
      Serial.print(ETH.macAddress());
      Serial.print(", IPv4: ");
      Serial.print(ETH.localIP());
      if (ETH.fullDuplex()) {
        Serial.print(", FULL_DUPLEX");
      }
      Serial.print(", ");
      Serial.print(ETH.linkSpeed());
      Serial.println("Mbps");
      eth_connected = true;
      break;
    case SYSTEM_EVENT_ETH_DISCONNECTED:
      Serial.println("ETH Disconnected");
      eth_connected = false;
      break;
    case SYSTEM_EVENT_ETH_STOP:
      Serial.println("ETH Stopped");
      eth_connected = false;
      break;
    default:
      break;
  }
}

void testClient(const char * host, uint16_t port) {
  Serial.print("\nconnecting to ");
  Serial.println(host);

  WiFiClient client;
  if (!client.connect(host, port)) {
    Serial.println("connection failed");
    return;
  }
  client.printf("GET / HTTP/1.1\r\nHost: %s\r\n\r\n", host);
  while (client.connected() && !client.available());
  while (client.available()) {
    Serial.write(client.read());
  }

  Serial.println("closing connection\n");
  client.stop();
}

void setup() {
  Serial.begin(115200);
  WiFi.onEvent(WiFiEvent);
  ETH.begin(ETH_ADDR, ETH_POWER_PIN, ETH_MDC_PIN, ETH_MDIO_PIN, ETH_TYPE, ETH_CLK_MODE);
  
  delay(1000); //coffee break;
  server.on("", HTTP_GET, handle);
  server.on("/", HTTP_GET, handle);
  server.begin();

}

void loop() {
  if (eth_connected) {
    testClient("google.com", 80);
    server.handleClient();
  }
  delay(1);
}

Debug Message

as above sketch

in handle:
url:http://192.168.4.1/?getnetwork
server.args() = 0 (wrong count)

url:http://192.168.4.1/?getnetwork=1&debug=true
server.args() = 2 (correct)

url:http://192.168.4.1/?getnetwork&debug=true
server.args() = 1 (wrong count)

Other Steps to Reproduce

No response

I have checked existing issues, online documentation and the Troubleshooting Guide

• I confirm I have checked existing issues, online documentation and Troubleshooting guide.

[SuGlider]

@PilnyTomas - Would you like to test it and investigate this issue? Let me know.

[PilnyTomas]

Hi @tsctrl, please modify the sketch to be complete - i.e. when I copy-paste it into Arduino IDE it will compile.

[tsctrl]

edited as per request and will compile

[PilnyTomas]

A web server needs some connection, there is no web connection in the provided sketch, and it crashes on startup.
Please provide a complete and minimal sketch that demonstrates the issue.
A modified example from the library can be used too.

Complete = compiles and starts properly after copy-paste (only change needed in code relates to SSID+PWD)
Minimal = only code needed to demonstrate the issue - issue demonstration does not require the rest of the system for which it is used.

[tsctrl]

updated with a complete and minimal sketch. does not require ssid and password

[PilnyTomas]

Can you please explain how did you manage to get working ETHernet on a bare ESP32 DevKitC?
I also don't understand why you mix WiFi and Ethernet.

Please provide a complete and minimal sketch that demonstrates the issue. A modified example from the library can be used too.

Complete = compiles and starts properly after copy-paste (only change needed in code relates to SSID+PWD)
Minimal = only code needed to demonstrate the issue - issue demonstration does not require the rest of the system for which it is used.

Please provide an example that is able to reproduce your issue with all other needed steps to reproduce.

[SuGlider]

The real question of this issue is:

refer example sketch as below. is this is valid result? opposite with
ESPAsyncWebServer libs request.params() count without equal= after ? as +1

example with WebServer:

url:http://192.168.4.1/?getnetwork
server.args() = 0 (wrong count)

url:http://192.168.4.1/?getnetwork=1&debug=true
server.args() = 2 (correct)

url:http://192.168.4.1/?getnetwork&debug=true
server.args() = 1 (wrong count)

This related to WebServer Library in Parsing.cpp -- it doesn't matter if it is using ETH or WiFi.