SSID names are padded with spaces at the end - making them all 31 characters long. about esp8266_beaconspam

[spacehuhn]

Thanks!
You're absolutely right. I will update the source code

Edit: I confused this with your second issue #15

The spaces are intentional. Without it, you need to copy a lot more bytes back and forth. Slowing the whole thing down.

What you changed now will create a faulty beacon frame. Saying the SSID is shorter than the packet is (because I made it always 32chars long).

A better alternative could be to fill it with \0 instead of spaces, but then you also get some crazy behaviour on some devices. WiFi implementations are weird and you can expect different scan results on different devices.

[neilbrookins]

OK, I agree. So, there is not a perfect solution. I guess to make the SSID field truly variable length, you'd have to do a copy of the bytes in the beaconPacket after the SSID parameter forward or back based on the new length needed. I wouldn't think that would take very long, but I haven't tested it to measure the impact.

I wonder if the sending of longer SSID with trailing space padding (always 32 bytes) is also slower than sending a shorter SSID? If so, perhaps some of the extra time overhead of the copying bytes for a variable length beacon packet is regained back by sending fewer bytes over the air?