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

Add BROKEN_NAT env variable to select Udp sender port workaround #501

Merged
merged 1 commit into from
Jun 30, 2018

Conversation

mvines
Copy link
Member

@mvines mvines commented Jun 30, 2018

My router needs BROKEN_NAT=1 this, seems like @CriesofCarrots's router does not (which is much more sane behavior). Need more data on other routers...

Paging issue #496

@mvines mvines requested a review from CriesofCarrots June 30, 2018 02:42
@mvines
Copy link
Member Author

mvines commented Jun 30, 2018

@garious I'm curious how your router behaves when you have a moment to test

@mvines mvines added the automerge Merge this Pull Request automatically once CI passes label Jun 30, 2018
@solana-grimes solana-grimes merged commit 903ec27 into solana-labs:master Jun 30, 2018
@mvines mvines deleted the nat branch July 8, 2018 03:10
jeffwashington pushed a commit to jeffwashington/solana that referenced this pull request Mar 29, 2024
…grams (solana-labs#501)

test-validator: clone upgradeable programs with slot 0
lijunwangs pushed a commit to lijunwangs/solana that referenced this pull request Apr 7, 2024
…ble programs (backport of solana-labs#501) (solana-labs#518)

Test Validator: Set deployment slot to `0` for cloned upgradeable programs (solana-labs#501)

test-validator: clone upgradeable programs with slot 0
(cherry picked from commit 4b0e7d6)

Co-authored-by: Joe C <joecaulfield29@yahoo.com>
willhickey pushed a commit that referenced this pull request Apr 8, 2024
…ble programs (backport of #501) (#518)

Test Validator: Set deployment slot to `0` for cloned upgradeable programs (#501)

test-validator: clone upgradeable programs with slot 0
(cherry picked from commit 4b0e7d6)

Co-authored-by: Joe C <joecaulfield29@yahoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants