Skip to content

Conversation

modwodmm
Copy link

@modwodmm modwodmm commented Oct 13, 2025

Fixed the jackson error

Closes #1314

@modwodmm modwodmm requested a review from a team as a code owner October 13, 2025 18:18
@CLAassistant
Copy link

CLAassistant commented Oct 13, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@Zabuzard Zabuzard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to be sure, did you run the code locally on your test server to verify that it works?
Because the problem only showed itself when actually running the code, not during compilation or in unit tests.

Comment on lines +79 to +82
exclude group: 'com.fasterxml.jackson.core'
exclude group: 'com.fasterxml.jackson.datatype'
exclude group: 'com.fasterxml.jackson.dataformat'
exclude group: 'com.fasterxml.jackson.module'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add an indentation (or run spotless)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, ran gradlew.bat run and it ran without any error and showed build successful

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add an indentation (or run spotless)

ran spotless as well

Copy link
Member

@Zabuzard Zabuzard Oct 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, ran gradlew.bat run and it ran without any error and showed build successful

thats not enough though. u need to actually run the bot, connect it to your local test server and for example use the /ping command successfully :)

the compilation worked before already as well.
and u need to check it with the updated dependency as the error doesnt occur beforehand anyways.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i guess spotless doesnt run on the gradle file then. either case, please add indents, thanks

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You also need to update the jackson dependency to provoke the problematic setup. The current versions never created any problems to begin with. See the ticket description again and the linked jackson update :)

@Zabuzard Zabuzard added bug Something isn't working priority: critical labels Oct 14, 2025
@modwodmm
Copy link
Author

Everything done. Changed the version and also tested it.
image

@modwodmm modwodmm closed this Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority: critical

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GitHub API and Open-AI crashes with Jackson update

3 participants