-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
correctly handle path escaping for connector IDs #2290
Conversation
Signed-off-by: Bob Callaway <bob.callaway@gmail.com>
Signed-off-by: Bob Callaway <bob.callaway@gmail.com>
@nabokihms any update on the release date for this? I appreciate any help in nudging this along. TIA! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Thank you a lot. I have only one moment to discuss, and then we will be able to move further with it.
As far as I can tell there are changes requested, so moving to next release. |
@bobcallaway @nabokihms any update about the status of this PR? From what I can tell there is a debate about the implementation. |
We can still accept the current implementation. The debate is about details o internal component implementation and only about cosmetics. @bobcallaway, I think we can accept this PR as is, yet rebasing the branch is still required. |
ok, i will rebase it then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution.
Fixes: #2289
Signed-off-by: Bob Callaway bob.callaway@gmail.com
Overview
What this PR does / why we need it
Special notes for your reviewer
Does this PR introduce a user-facing change?