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

udp/tcp/dtls: Done() signalizes that connection is not more processed. #247

Merged
merged 2 commits into from
Sep 8, 2021
Merged

udp/tcp/dtls: Done() signalizes that connection is not more processed. #247

merged 2 commits into from
Sep 8, 2021

Conversation

jkralik
Copy link
Member

@jkralik jkralik commented Sep 8, 2021

No description provided.

@jkralik jkralik force-pushed the jkralik/feature/signalizingDone branch from 69b3bad to dced452 Compare September 8, 2021 07:37
@jkralik jkralik force-pushed the jkralik/feature/signalizingDone branch from dced452 to 26c8e2a Compare September 8, 2021 07:51
der-dano
der-dano reviewed Sep 8, 2021
View reviewed changes
Copy link
Collaborator

@der-dano der-dano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't we add a simple test case for this new flag?

@jkralik
Copy link
Member Author

jkralik commented Sep 8, 2021

Shouldn't we add a simple test case for this new flag?

Done

@jkralik jkralik merged commit 4a904cb into master Sep 8, 2021
@jkralik jkralik deleted the jkralik/feature/signalizingDone branch September 8, 2021 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@der-dano der-dano der-dano approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jkralik @der-dano