[lwip-devel] [patch #9525] httpd: add Websocket support, fix lwip_strnicmp

[Simon Goldschmidt]

Follow-up Comment #4, patch #9525 (project lwip):

I really don't know if this fits. Anyway, a few generic comments:
- don't add third party code without a license that fits. I can't even see a
license at all for base64 and sha1
- when using sha1, use it like ppp does (polarssl/mbedtls)
- base64 encoding is used in smtp, too. We need to combine this instead of
adding it a second time. Again, watch out for licensing issues here!

As to the actual web socket implementation: I won't add it like this to
httpd.c. That file is too big and unreadable like it is now, adding yet more
code would be bad. You might try to add a httpd_websockets.c file and come off
with minimal changes to integrate this into httpd.c

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?9525>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/