Skip to content

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

Merged
miri64 merged 1 commit intoRIOT-OS:masterfrom
haukepetersen:opt_gcoap_passudpeponresp
Jul 31, 2017
Merged

net/gcoap: added remote sock ep to resp handler cb#7407
miri64 merged 1 commit intoRIOT-OS:masterfrom
haukepetersen:opt_gcoap_passudpeponresp

Conversation

@haukepetersen
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

rebased

Copy link
Copy Markdown
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