8000 Handle ssh:// URLs that doesn't contain git@ by kad · Pull Request #110 · jenkinsci/github-plugin · GitHub
[go: up one dir, main page]

Skip to content

Handle ssh:// URLs that doesn't contain git@ #110

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

Merged
merged 2 commits into from
Feb 19, 2016

Conversation

kad
Copy link
Contributor
@kad kad commented Feb 19, 2016

It is possible to specify and use github.com URLs as
ssh://github.com/username/repo, as credentials used
for SCM configuration usually include username.
This change would allow to properly parse such URLs.

Review on Reviewable

It is possible to specify and use github.com URLs as
ssh://github.com/username/repo, as credentials used
for SCM configuration usually include username.
This change would allow to properly parse such URLs.
@jenkinsadmin
Copy link
Member

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

@KostyaSha
Copy link
Member

LGTM, @lanwen ?

@kad
Copy link
Contributor Author
kad commented Feb 19, 2016

After second look, it would be good to update also test: github-plugin/src/test/java/com/coravy/hudson/plugins/github/GitHubRepositoryNameTest.java. I'll update PR in few hours.

@KostyaSha
Copy link
Member

Yes, text case required.

@kad
Copy link
Contributor Author
kad commented Feb 19, 2016

Test case updated.

lanwen added a commit that referenced this pull request Feb 19, 2016
Handle ssh:// URLs that doesn't contain git@
@lanwen lanwen merged commit d49f0cb into jenkinsci:master Feb 19, 2016
@lanwen
Copy link
Member
lanwen commented Feb 19, 2016

thanks!

@lanwen
Copy link
Member
lanwen commented Feb 22, 2016

will be released as 1.17.1 soon

@KostyaSha
Copy link
Member

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0