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

Fix sending UDP response using new SDK 1.4.0 #750

Merged
merged 1 commit into from
Nov 11, 2015
Merged

Fix sending UDP response using new SDK 1.4.0 #750

merged 1 commit into from
Nov 11, 2015

Conversation

lukaszo
Copy link
Contributor

@lukaszo lukaszo commented Nov 10, 2015

From release notes of new SDK
Modify the espconn pointer in the receive callback of UDP. Parameters remote_ip and remote_port in it are the remote IP and port set by espconn_create. If users want to obtain IP and ports of the current sender, please call espconn_get_connection_info to get relevant information.

This it copy paste from net module: https://github.com/nodemcu/nodemcu-firmware/blob/dev/app/modules/net.c#L1080-L1088

@jmattsson
Copy link
Member

I believe we might need the same fix over in enduser_setup.c too then. @robertfoss do you want to weigh in?

jmattsson pushed a commit that referenced this pull request Nov 11, 2015
Fix sending CoAP UDP response using new SDK 1.4.0
@jmattsson jmattsson merged commit 340edbb into nodemcu:dev Nov 11, 2015
@TerryE
Copy link
Collaborator

TerryE commented Nov 11, 2015

Also in net udp server probably and see #223

PS. Duurrrhh. Johny fixed this a month ago.

@robertfoss
Copy link
Contributor

jmattson: Having a look.

robertfoss added a commit to robertfoss/nodemcu-firmware that referenced this pull request Nov 12, 2015
jmattsson pushed a commit that referenced this pull request Nov 17, 2015
 Added fix from #750, to access the IP of a sender of a UDP packet, to fix SDK 1.4.0 compability.
@hreintke hreintke mentioned this pull request Dec 5, 2015
@lukaszo lukaszo deleted the coap-deb branch March 1, 2019 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants