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

net/gcoap: added remote sock ep to resp handler cb #7407

Merged
merged 1 commit into from
Jul 31, 2017

Conversation

haukepetersen
Copy link
Contributor

When receiving a response to a request send our earlier, it is in some cases essential, to know something about who send the reponse. This is needed for example when sending requests to multicast addresses to be able to identify the responding node.

@haukepetersen haukepetersen added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jul 25, 2017
@haukepetersen haukepetersen added this to the Release 2017.10 milestone Jul 25, 2017
@kb2ma
Copy link
Member

kb2ma commented Jul 30, 2017

ACK -- simple and straightforward.

When receiving a response to a request send our earlier, it is in
some cases essential, to know something about who send the reponse.
This is needed for example when sending requests to multicast
addresses to be able to react to different incoming responses.
@haukepetersen haukepetersen force-pushed the opt_gcoap_passudpeponresp branch from 57ef845 to 2290a9f Compare July 31, 2017 09:11
@haukepetersen
Copy link
Contributor Author

rebased

Copy link
Member

@miri64 miri64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Proxy-ACK

@miri64 miri64 merged commit ef9007b into RIOT-OS:master Jul 31, 2017
@haukepetersen haukepetersen deleted the opt_gcoap_passudpeponresp branch July 31, 2017 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants