Skip to content
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.

Sign up for GitHub

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

Jump to bottom

improved dtls construction, fix resource leak on recv_loop, update version #100

Merged
merged 3 commits into from
Apr 9, 2024
Merged

improved dtls construction, fix resource leak on recv_loop, update version #100

merged 3 commits into from
Apr 9, 2024

Conversation

osobiehl
Copy link
Contributor

@osobiehl osobiehl commented Apr 7, 2024

changelog:

  • DTLS can now be used with a custom transport as long as it implements Conn + Send + Sync
  • an "on drop" hook for DTLS may now be used when the client is dropped -> may be used if you want to resume a session, also tested
  • Change how DTLS connections are made so users can use their own types, as well as add existing state, also tested
  • rename "Transport" to "ClientTransport" since it is only used by the client
  • ClientTransport no longer cares about a Sockaddr return type since it's not needed for the client
  • fix issue where a connection would never be dropped because recv_loop would run forever (changed Arc to Weak)
  • update version to 0.17

osobiehl added 3 commits April 6, 2024 13:55
move DTLS to be able to be used with an own struct implementing Conn …
9bd773a 
…from dtls
allow different implementations of connection for DTLS
d80c56c 
remove need for client transport to need an address
rename transport to ClientTransport since only client uses it
fixed leak cycle issue on client where connection is never dropped
update version
51ed9dd
@coveralls
Copy link

Coverage Status

coverage: 82.169% (+0.4%) from 81.784%
when pulling 51ed9dd on osobiehl:feature/jb/improved-dtls-construction
into 8825bc3 on Covertness:master.

@Covertness Covertness merged commit c120146 into Covertness:master Apr 9, 2024
3 checks passed
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@osobiehl @coveralls @Covertness