[riot-notifications] [RIOT-OS/RIOT] sys/net/application_layer/nanocoap: Add path prefix option (#11098)

Kaspar Schleiser notifications at github.com
Mon Mar 4 22:08:52 CET 2019


kaspar030 commented on this pull request.



> @@ -323,7 +323,15 @@ ssize_t coap_handle_req(coap_pkt_t *pkt, uint8_t *resp_buf, unsigned resp_buf_le
             continue;
         }
 
-        int res = strcmp((char *)uri, resource->path);
+        int res;

Maybe at that point a custom strcmp would make sense. It could do strncmp, but if it arrives at the end of resource->path, if up to that point the strings matched and the current path position is '/', it considers that as a match.
Unless there is a use cases for prefixes that are not a whole subtree, that would remove the need for the method flag altogether.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11098#discussion_r262243118
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190304/40f92986/attachment.html>


More information about the notifications mailing list