Skip to content
This repository has been archived by the owner on Jan 20, 2025. It is now read-only.

Bugfix: Make library work with ESP32 v3.1.0 #189

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

dalathegreat
Copy link
Contributor

What

The changes introduced in EPS32 v3.1.0 causes a runtime crash when trying to use this AsyncTCP library

assert failed: tcp_alloc /IDF/components/lwip/lwip/src/core/tcp.c:1851 (Required to lock TCPIP core functionality!)

How

We fix this by applying the fix in espressif/arduino-esp32#10526 , suggested by mathieucarbou

@Lukaswnd
Copy link
Contributor

Lukaswnd commented Jan 3, 2025

In general it works for me. But you forgot to commit the AsyncTCP.h you added 2 functions, which also need to be added in the class declaration in the header.
And you should check backward compability

@mathieucarbou
Copy link
Contributor

mathieucarbou commented Jan 16, 2025

@dalathegreat : fyi the initial fix was improved.

you can have a look at the community maintained forks here where several people are collaborating:

Ideally the author would need to merge all the changes from these forks in his own repos if he intends one day to resume the work and maintenance of these repos, and give collaborator access to people to ensure that these won't be dead again.

@dalathegreat
Copy link
Contributor Author

Thanks @mathieucarbou , I have switched my Battery-Emulator project over to using your excellent forks of ESPAsyncWebServer & AsyncTCPsock!

I will close this PR since it obviosuly is not a repo that is taking in new contributions!

@mathieucarbou
Copy link
Contributor

Thanks @mathieucarbou , I have switched my Battery-Emulator project over to using your excellent forks of ESPAsyncWebServer & AsyncTCPsock!

We are maintaining AsyncTCP, not AsyncTCPSock ;-)

@me-no-dev me-no-dev reopened this Jan 20, 2025
@me-no-dev me-no-dev merged commit ca0ead0 into me-no-dev:master Jan 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants