-
Notifications
You must be signed in to change notification settings - Fork 638
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
Credentials-Helper and --rootPath #1060
Comments
Ah, you are absolutely right. It seems like you have a solid fix for it and feel free to send me a PR. If not I will create one later this week. Thanks |
I think this can be closed right? Fixed with #1061 right? |
Hey, yes, this is fixed with #1061, but I think there is still a problem with |
Ah... do you mean #1061 helped but it didn't fully fixed it? yes can you please open a new ticket with the details? thanks! |
Hey,
I discoverd that the
bin/credentials-helper
ignores the--rootPath
, because the request urlhttp://localhost:${port}/api/credentials
in the credentials-helper#L14 is without therootPath
.Test-Case
Run the
ungit
command with the--rootPath
like:Open http://localhost:4000/ungit/ and try to clone a repository. The clone operation fails with the followring response:
Solution
Just pass the
rootPath
to thebin/credentials-helper
, likesocketId
ororigin
.My-Quick-Fix
I add a environment variable
ROOT_PATH
to the url for thebin/credentials-helper
.Example:
The text was updated successfully, but these errors were encountered: