Skip to content

Added destroy callback for transport services#612

Merged
soffokl merged 1 commit intomasterfrom
cleanup-transport-on-destroy
Dec 14, 2018
Merged

Added destroy callback for transport services#612
soffokl merged 1 commit intomasterfrom
cleanup-transport-on-destroy

Conversation

@soffokl
Copy link
Copy Markdown
Member

@soffokl soffokl commented Dec 13, 2018

No description provided.

@soffokl soffokl added this to the Keliukis (0.5) milestone Dec 13, 2018
@soffokl soffokl self-assigned this Dec 13, 2018
@soffokl soffokl requested a review from vkuznecovas December 13, 2018 05:59
@soffokl soffokl force-pushed the wireguard-multi-connection branch from b1c0435 to 932b943 Compare December 13, 2018 08:22
Comment thread services/wireguard/endpoint/endpoint.go
Comment thread services/wireguard/endpoint/kernelspace/client.go Outdated
@soffokl soffokl force-pushed the wireguard-multi-connection branch from 774c8f6 to e27a470 Compare December 13, 2018 11:10
@soffokl soffokl changed the base branch from wireguard-multi-connection to master December 13, 2018 14:27
@soffokl soffokl force-pushed the cleanup-transport-on-destroy branch from 30d71f9 to 3b735fd Compare December 13, 2018 14:27
@zolia zolia closed this Dec 13, 2018
@soffokl
Copy link
Copy Markdown
Member Author

soffokl commented Dec 13, 2018

@zolia this one still in progress.

@soffokl soffokl reopened this Dec 13, 2018
@zolia zolia closed this Dec 13, 2018
@soffokl soffokl reopened this Dec 13, 2018
@soffokl soffokl requested a review from vkuznecovas December 13, 2018 15:31
@soffokl soffokl force-pushed the cleanup-transport-on-destroy branch from e11a662 to 100a85a Compare December 13, 2018 15:32
@soffokl
Copy link
Copy Markdown
Member Author

soffokl commented Dec 13, 2018

@vkuznecovas @zolia @tadovas please take a look on this one.
Left only destroyCallback without renaming configProvider

@soffokl soffokl merged commit 7749a27 into master Dec 14, 2018
@soffokl soffokl deleted the cleanup-transport-on-destroy branch December 14, 2018 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants