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 job import of project with whitespace #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

romainjacquet
Copy link

  • fix job import with a name containing whitespace by properly encode the url

  • display the HTTP error code in case of the import failed to help debugging

debugging.

fix job import with a name containing whitespace
@icereed
Copy link
Contributor

icereed commented May 22, 2019

Good job! How did you test that?

@romainjacquet
Copy link
Author

I have tested it during a migration of jobs that contains space at my office. Adding a test is not simple because for the moment, the function ImportJob has no test. I think that the commit is small enough to be checked without adding a test.
What do you think?

@icereed
Copy link
Contributor

icereed commented Jun 5, 2019

I think it's a quite cool contribution, thank you! @mlabouardy would you like to merge this?

@grootech
Copy link

grootech commented Mar 4, 2020

how can i import to jenkins which is working with my Gsuiute SSO ? it always says Unauthorized 401

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.

3 participants