Skip to content
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

[WIP] Provide a migration path from ESPAsyncWebServer (PoC) #115

Draft
wants to merge 1 commit into
base: v2-dev
Choose a base branch
from

Conversation

mathieucarbou
Copy link
Collaborator

@mathieucarbou mathieucarbou commented Aug 7, 2024

This PR is a PoC for #110

Project PR Status Components Comment
mathieucarbou/MycilaESPConnect#25 🚧 Filter, HTTP
https://github.com/mathieucarbou/MycilaWebSerial 🚧 Filter, Websocket (text based) High streaming rate of log lines of random length
https://github.com/ayushsharma82/WebSerial 🚧 Filter, Websocket (binary based) High streaming rate of log lines of random length
https://github.com/ayushsharma82/ElegantOTA 🚧 Filter, HTTP
https://github.com/ayushsharma82/ESP-DASH 🚧 Filter, Websocket (text based) Potentially big messages, high send rate for real-time dashboards

🚧: WIP or TODO
✅: Done and tested
❌: Blocked or not possible

I do not have any event source example (I don't like event source lol) so someone will have to contribute this part.

@mathieucarbou mathieucarbou force-pushed the espasyncws branch 3 times, most recently from 3da0872 to e936e0a Compare August 7, 2024 21:38
@hoeken
Copy link
Owner

hoeken commented Aug 7, 2024

I think I see what you're going for here and I'm open to it.

Can you explain a bit more about why you changed from by pointer to by reference? My C++ is a bit rusty in this area. Is it to make the syntax more similar to ESPAsync?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants