-
Notifications
You must be signed in to change notification settings - Fork 1
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
Implement support for CoAP over tcp #12
Conversation
Implement the missing structures. Add tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two minor comments regarding documentation that references UDP instead of TCP, the rest looks fine to me.
Thanks for the PR! Regarding the test failures: It seems that rust-lang/rust#109044 is the cause. |
Fix copy/paste comments. Everything should reffer to the TCP functionality.
I have fixed the comments. The changes are pushed.
Glad that I could help. |
I've rebased the branch on main, so that it includes @pulsastrix switch to nightly toolchain. |
@pulsastrix @falko17 I guess this PR should now be ready to be merged, right? Or does the coverage job need to be fixed first? |
@JKRhb the coverage job failure is due to the fact that this PR comes from a fork, which causes the CI pipeline to have insufficient permissions to create comments in the main repository. |
This pull request implements CoAP over TCP.