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

The test connection button on Jenkins' configure page throws Client error: CAMEL_CASE_TO_LOWER_CASE_WITH_UNDERSCORES #1605

Closed
aah9 opened this issue Jan 2, 2024 · 6 comments · Fixed by #1606
Labels
bug For changelog: Minor bug. Will be listed after features

Comments

@aah9
Copy link

aah9 commented Jan 2, 2024

Jenkins and plugins versions report

Environment
Jenkins: 2.426.2
OS: Linux - 4.14.330-250.540.amzn2.x86_64
Java: 11.0.20 - Red Hat, Inc. (OpenJDK 64-Bit Server VM)
---
git:5.2.1
git-client:4.6.0
github:1.37.3.1
github-api:1.318-461.v7a_c09c9fa_d63
github-branch-source:1767.va_7d01ea_c7256
gitlab-plugin:1.7.18
jackson2-api:2.16.1-373.ve709c6871598

What Operating System are you using (both controller, and any agents involved in the problem)?

Amazon Linux 2

Reproduction steps

  1. Install latest version of jackson2-api plugin (2.16.1-373.ve709c6871598) + gitlab-plugin
  2. Navigate to configure page
  3. Setup a gitlab connection
  4. Press Test Connection button

Expected Results

Success message should display.
Expected-GitLab-Connection-Test

Actual Results

Error msg displays: Client error: CAMEL_CASE_TO_LOWER_CASE_WITH_UNDERSCORES
Actual-GitLab-Connection-Test

Anything else?

Related to JENKINS-72478 Jackson2 API 2.16.1 removes Java PropertyNamingStrategy aliases

Are you interested in contributing a fix?

No response

@aah9 aah9 added the bug For changelog: Minor bug. Will be listed after features label Jan 2, 2024
@aah9
Copy link
Author

aah9 commented Jan 2, 2024

I think the fix is switching from CAMEL_CASE_TO_LOWER_CASE_WITH_UNDERSCORES to SNAKE_CASE:

FasterXML/jackson-databind@d7a1efc

@MarkEWaite
Copy link
Contributor

I think the fix is switching from CAMEL_CASE_TO_LOWER_CASE_WITH_UNDERSCORES to SNAKE_CASE:

FasterXML/jackson-databind@d7a1efc

Thanks for reporting that @aah9 ! Would you like to submit the pull request to fix it?

@bruderol
Copy link
Contributor

bruderol commented Jan 2, 2024

Same problem here - gitlab plugin is currently not working like this

@MarkEWaite
Copy link
Contributor

Same problem here - gitlab plugin is currently not working like this

Thanks for confirming. Would you like to submit the pull request to fix it? You have an environment where you can test it more rapidly than I can configure a test environment.

bruderol added a commit to bruderol/gitlab-plugin that referenced this issue Jan 2, 2024
…WITH_UNDERSCORES by new SNAKE_CASE constant to work with newer Jersey plugin (jenkinsci#1605)
@bruderol
Copy link
Contributor

bruderol commented Jan 2, 2024

I filed a Pull request - let me know how I can easily test it - i have no clue how to do so (how would I get the fixed plugin on my jenkins ?)

@MarkEWaite
Copy link
Contributor

MarkEWaite commented Jan 2, 2024

I filed a Pull request - let me know how I can easily test it - i have no clue how to do so (how would I get the fixed plugin on my jenkins ?)

I've uploaded a zip file of the plugin build. Download it to your local computer, expand the zip file, then use the "Advanced" section of the Jenkins plugin manager page to upload the gitlab-plugin.hpi file. Restart your Jenkins controller and you should see the new SNAPSHOT version displayed in the plugin manager list of installed plugins.

gitlab-plugin.zip

MarkEWaite added a commit to MarkEWaite/gitlab-plugin that referenced this issue Jan 2, 2024
Needed for the Jackson 2.16.1-373.ve709c6871598 release that removes
Jackson 2 API constants used by this plugin.

jenkinsci#1605 notes that the
CAMEL_CASE_TO_LOWER_CASE_WITH_UNDERSCORES constant needs to be replaced
with SNAKE_CASE.

FasterXML/jackson-databind#4162
removed those deprecated aliases in
FasterXML/jackson-databind@d7a1efc

This change is preparation for a dependency on Jackson 2 API 2.16.1.
MarkEWaite added a commit that referenced this issue Jan 3, 2024
Needed for the Jackson 2.16.1-373.ve709c6871598 release that removes
Jackson 2 API constants used by this plugin.

#1605 notes that the
CAMEL_CASE_TO_LOWER_CASE_WITH_UNDERSCORES constant needs to be replaced
with SNAKE_CASE.

FasterXML/jackson-databind#4162
removed those deprecated aliases in
FasterXML/jackson-databind@d7a1efc

This change is preparation for a dependency on Jackson 2 API 2.16.1.
MarkEWaite pushed a commit that referenced this issue Jan 3, 2024
…WITH_UNDERSCORES by new SNAKE_CASE constant to work with newer Jersey plugin (#1605) (#1606)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug For changelog: Minor bug. Will be listed after features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants