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.

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

Update innernet to v1.5.0 #434

Closed
wants to merge 37 commits into from
Closed

Update innernet to v1.5.0 #434

wants to merge 37 commits into from

Conversation

plan5
Copy link

@plan5 plan5 commented Sep 19, 2021

The package builds fine, I'm not able to test right now. For testers: make sure to backup your sqlite database as recommended here:
https://github.com/tonarino/innernet/releases/tag/v1.5.0

This update is supposed to improve NAT traversal.

plan5 and others added 30 commits March 22, 2021 22:41
Co-authored-by: Nathaniel van Diepen <Eeems@users.noreply.github.com>
Co-authored-by: Nathaniel van Diepen <Eeems@users.noreply.github.com>
Co-authored-by: Nathaniel van Diepen <Eeems@users.noreply.github.com>
…into testing"

This reverts commit 324dc3a, reversing
changes made to 127565d.
@Eeems Eeems changed the base branch from stable to testing September 19, 2021 23:15
@Eeems
Copy link
Member

Eeems commented Sep 19, 2021

Duplicate of #432 ? Although you do have some changes that my PR doesn't have.

@plan5
Copy link
Author

plan5 commented Sep 20, 2021

Duplicate of #432 ? Although you do have some changes that my PR doesn't have.

Ah, yes it's a duplicate. Mine contains a corrected path for the to-be-patched build file as I see it. I don't mind updating that one or keeping this one.

@Eeems
Copy link
Member

Eeems commented Sep 20, 2021

It might be best to use #432 since this PR seems to have more noise.

@plan5
Copy link
Author

plan5 commented Sep 20, 2021

It might be best to use #432 since this PR seems to have more noise.
That's right. I'm closing this one.

@plan5 plan5 closed this Sep 20, 2021
@plan5 plan5 mentioned this pull request Sep 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants