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

[fix] windows reload should check via #1166

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

sohunjug
Copy link

When using this library for secondary development as a dependency, or when invoking a reload through sshd, if you modify unsafe_route corresponding to via, please delete the old information.

Copy link

Thanks for the contribution! Before we can merge this, we need @sohunjug to sign the Salesforce Inc. Contributor License Agreement.

@wadey
Copy link
Member

wadey commented Jun 17, 2024

Line 26 a at the top of this method says:

// Via is ignored since that is only consumed within nebula itself

So is that comment incorrect?

@sohunjug
Copy link
Author

Line 26 a at the top of this method says:

// Via is ignored since that is only consumed within nebula itself

So is that comment incorrect?

windows

route print

when i want change unsafe_route: via, reload will create multi routes with different via

@wadey wadey added the NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. label Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla:signed NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants