Skip to content

Add client dtls connection ID (2.). - #149

Closed
boaks wants to merge 2 commits into
eclipse-tinydtls:mainfrom
boaks:client_cid
Closed

Add client dtls connection ID (2.).#149
boaks wants to merge 2 commits into
eclipse-tinydtls:mainfrom
boaks:client_cid

Conversation

@boaks

@boaks boaks commented Jun 25, 2022

Copy link
Copy Markdown
Contributor

Simple client side implementation indicates support and uses the cid of
the server, when negotiated by that.

Signed-off-by: Achim Kraus achim.kraus@cloudcoap.net

@boaks boaks mentioned this pull request Jun 25, 2022
Closed
@boaks boaks mentioned this pull request Aug 9, 2022
Open
@boaks
boaks force-pushed the client_cid branch 4 times, most recently from fa939b1 to 0af8db6 Compare January 25, 2023 09:10
@mrdeep1 mrdeep1 mentioned this pull request Feb 6, 2023
Closed
@boaks
boaks force-pushed the client_cid branch 2 times, most recently from 4becf0b to 0272b27 Compare March 11, 2023 11:17
@boaks
boaks force-pushed the client_cid branch 2 times, most recently from d8e717b to fb7a4aa Compare April 24, 2023 09:35
@boaks
boaks force-pushed the client_cid branch 2 times, most recently from 3dabb24 to 4b60a6b Compare July 14, 2023 14:35
@boaks boaks added the available on develop Mark PRs (pre-)available only on develop label Feb 7, 2024
@boaks
boaks force-pushed the client_cid branch 2 times, most recently from 7bc46d1 to db5b584 Compare August 28, 2024 14:40
@mrdeep1

mrdeep1 commented Sep 16, 2024

Copy link
Copy Markdown
Contributor

It would be good if a server side API could be added where the calling application can define the CID to use so that it can be tracked by the application to change the tuple to send back any response on.

@boaks

boaks commented Sep 16, 2024

Copy link
Copy Markdown
Contributor Author

server side API

As for now, tinydtls CID only supports the client side. To be more precise, the client only indicates to support CID, but doesn't use a CID on it's own side. For the next time I will not able to spend the required time to implement a server side. If that changes, the proposed API will make a lot of sense.

boaks added 2 commits May 19, 2025 11:47
@boaks
Add client dtls connection ID.
7cf7699 
Simple client side implementation indicates support and uses the cid of
the server, when negotiated by that.

Signed-off-by: Achim Kraus <achim.kraus@cloudcoap.net>
@boaks
dtls-client.c: add support_cid option.
f5e0ca7 
Signed-off-by: Achim Kraus <achim.kraus@cloudcoap.net>
@boaks boaks closed this May 19, 2025
@mrdeep1

mrdeep1 commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

@boaks Any chance of getting this (or equivalent) merged into main?

@boaks

boaks commented Oct 16, 2026

Copy link
Copy Markdown
Contributor Author

If I remember it well, we want to try to get the most other PRs merged before adding CID.
Until that, CID is provided as feature/branch, see feature/connection_id. (Sometimes it will need some time to get updated :-) ) .

@boaks

boaks commented Oct 16, 2026

Copy link
Copy Markdown
Contributor Author

@obgm

Should we change that approach? The we may convert the feature branch into a PR.

@obgm

obgm commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

There is no harm in having a feature branch being a PR for main at the same time. So, if there is demand for CID, just go ahead opening a PR.

@mrdeep1

mrdeep1 commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

just go ahead opening a PR.

Just done this with #302.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

available on develop Mark PRs (pre-)available only on develop

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@boaks @mrdeep1 @obgm