examples/dtls-sock: fix credman capacity #20217
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fabian18
approved these changes
Jan 6, 2024
There was a problem hiding this comment.
You are right. Without it we cannot add the credentials.
But somehow the handshake does not work when I test on same54-xpro with Ethernet or with native. The server receives the first hello request but the client seems to be busy with retransmissions (not sure) and the shell is blocked.
2024-01-06 11:58:32,882 # dtlss start
> dtlsc ::1 hello
2024-01-06 11:58:39,295 # dtlsc ::1 hello
2024-01-06 11:58:39,298 # cannot create peer
2024-01-06 11:58:39,301 # error while handling handshake packet
2024-01-06 11:58:39,303 # 80 invalidate peer
2024-01-06 11:58:39,307 # received fatal alert, peer has been invalidated
|
I had the same issue on AVR :-/ At least we know that this is not an architecture specific issue |
It should be possible to test DTLS via `::1`, but this requires 4 slots in the credman instead of just two.
bff4269 to
3f57b8c
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Contribution description
It should be possible to test DTLS via
::1, but this requires 4 slots in the credman instead of just two.Testing procedure
Running both server and client on the same app should now succeed; or at least no longer fail when adding the credentials.
Issues/PRs references
Found while testing #20196