Skip to content

Fix for when user enters a badly formed github url for repo#32

Merged
samrocketman merged 1 commit into
jenkinsci:masterfrom
alexrothenberg:fix-npe-bug-whith-badly-formatted-github-repo-url
Jul 6, 2015
Merged

Fix for when user enters a badly formed github url for repo#32
samrocketman merged 1 commit into
jenkinsci:masterfrom
alexrothenberg:fix-npe-bug-whith-badly-formatted-github-repo-url

Conversation

@alexrothenberg
Copy link
Copy Markdown
Contributor

Previously it threw a NullPointerException which made the jenkins UI unusable
Now it treats that project as forbidden and will not show up (somewhat confusing as it "fails silently" but not sure what would be better)

Previously it threw a NullPointerException which made the jenkins UI unusable
Now it treats that project as forbidden and will not show up (somewhat confusing as it "fails silently" but not sure what would be better)
@jenkinsadmin
Copy link
Copy Markdown
Member

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

@alexrothenberg
Copy link
Copy Markdown
Contributor Author

Fixes [JENKINS-24383]

@steve-jansen
Copy link
Copy Markdown

@amccarty
Copy link
Copy Markdown

Any chance of pulling this sometime soon?

@tjcorr
Copy link
Copy Markdown

tjcorr commented May 14, 2015

+1 @mocleiri @skottler can someone please get this merged?

@samrocketman samrocketman merged commit 1d3f6bd into jenkinsci:master Jul 6, 2015
samrocketman added a commit that referenced this pull request Jul 6, 2015
formed github url for repo
@steve-jansen
Copy link
Copy Markdown

thank you @samrocketman !

@alexrothenberg
Copy link
Copy Markdown
Contributor Author

woot @samrocketman 🚀

@amccarty
Copy link
Copy Markdown

amccarty commented Jul 6, 2015

Thanks @samrocketman

@samrocketman
Copy link
Copy Markdown
Member

😄

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants