{"action":"created","issue":{"url":"https://api.github.com/repos/nmap/nmap/issues/369","repository_url":"https://api.github.com/repos/nmap/nmap","labels_url":"https://api.github.com/repos/nmap/nmap/issues/369/labels{/name}","comments_url":"https://api.github.com/repos/nmap/nmap/issues/369/comments","events_url":"https://api.github.com/repos/nmap/nmap/issues/369/events","html_url":"https://github.com/nmap/nmap/pull/369","id":149039865,"number":369,"title":"Initial submission of CoAP-related code to the Nmap project.","user":{"login":"mogigoma","id":130862,"avatar_url":"https://avatars.githubusercontent.com/u/130862?v=3","gravatar_id":"","url":"https://api.github.com/users/mogigoma","html_url":"https://github.com/mogigoma","followers_url":"https://api.github.com/users/mogigoma/followers","following_url":"https://api.github.com/users/mogigoma/following{/other_user}","gists_url":"https://api.github.com/users/mogigoma/gists{/gist_id}","starred_url":"https://api.github.com/users/mogigoma/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mogigoma/subscriptions","organizations_url":"https://api.github.com/users/mogigoma/orgs","repos_url":"https://api.github.com/users/mogigoma/repos","events_url":"https://api.github.com/users/mogigoma/events{/privacy}","received_events_url":"https://api.github.com/users/mogigoma/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-04-18T04:33:11Z","updated_at":"2016-09-08T20:59:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nmap/nmap/pulls/369","html_url":"https://github.com/nmap/nmap/pull/369","diff_url":"https://github.com/nmap/nmap/pull/369.diff","patch_url":"https://github.com/nmap/nmap/pull/369.patch"},"body":"I am submitting a new port definition, service probe, discovery script, and protocol library for CoAP, an Internet of Things HTML-like protocol. CoAP is standardized and has many implementations and public test servers. Along with MQTT, it appears to be one of the most popular protocols in its domain.\r\n\r\nI corrected what appeared to be an error in the `unittest` library's own unit tests. I have also included a new test method `unittest.identical` which is useful for performing deep comparisons of arbitrary Lua objects. This method tests the existence, type, length, and value of all objects recursively. Its existence allows the unit testing of the packet building and parsing features, many of which return nested tables.\r\n\r\nSince this protocol has many features which I found difficult to get just right, the library is very hierarchical, structured as sort of a tree where the leaves are usually `build()` or `parse()` functions. Points of complexity that could certainly use review are:\r\n\r\n* `COAP.header.options.build()`\r\n* `COAP.header.options.delta_length.build()`\r\n* `COAP.header.options.delta_length.parse()`\r\n* `COAP.payload.application_link_format.parse()`\r\n\r\nThe following command lines can be used to test multiple public test servers:\r\n\r\n* `nmap -p U:5683 -sU --script coap-resources coap.me`\r\n* `nmap -p U:5683 -sU --script coap-resources californium.eclipse.org`\r\n\r\nThe script output is a bit strange, having blank lines everywhere. I have not been able  to correct this, and have also failed to get the example code in `stdnse.format_output()` working as documented. Unsure if this is my failure or the library's, but assuming it's my fault for now.\r\n\r\nIf you have any questions or guidance, I will do my best to respond promptly."},"comment":{"url":"https://api.github.com/repos/nmap/nmap/issues/comments/245739031","html_url":"https://github.com/nmap/nmap/pull/369#issuecomment-245739031","issue_url":"https://api.github.com/repos/nmap/nmap/issues/369","id":245739031,"user":{"login":"mogigoma","id":130862,"avatar_url":"https://avatars.githubusercontent.com/u/130862?v=3","gravatar_id":"","url":"https://api.github.com/users/mogigoma","html_url":"https://github.com/mogigoma","followers_url":"https://api.github.com/users/mogigoma/followers","following_url":"https://api.github.com/users/mogigoma/following{/other_user}","gists_url":"https://api.github.com/users/mogigoma/gists{/gist_id}","starred_url":"https://api.github.com/users/mogigoma/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mogigoma/subscriptions","organizations_url":"https://api.github.com/users/mogigoma/orgs","repos_url":"https://api.github.com/users/mogigoma/repos","events_url":"https://api.github.com/users/mogigoma/events{/privacy}","received_events_url":"https://api.github.com/users/mogigoma/received_events","type":"User","site_admin":false},"created_at":"2016-09-08T20:59:42Z","updated_at":"2016-09-08T20:59:42Z","body":"No, it won't support it as-is. I tried to put things in place so that if DTLS support gets added it would start working. Port numbers and such. Feel free to remove if that's jumping the gun."}}
{"actor":{"display_login":"mogigoma"}}