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 crash on app start/API use on SDK <26 #2497

Merged
merged 1 commit into from
May 1, 2022

Conversation

jpelgrom
Copy link
Member

@jpelgrom jpelgrom commented May 1, 2022

Summary

Fixes #2465, fixes #2477

FasterXML/jackson-databind#3412 results in a crash when the app tries to use the Jackson databind library on devices with API level < 26 (= basically any API call, so from the moment the app starts and tries to contact the server). Manually specify the jackson-databind dependency with the latest version that includes a fix for now, Gradle will choose the highest version which solves the problem.

Screenshots

n/a

Link to pull request in Documentation repository

n/a

Any other notes

 - Manually specify databind version to fix FasterXML/jackson-databind#3412, as the Kotlin module still depends on the unfixed version and hasn't had a new release yet
Copy link
Member

@dshokouhi dshokouhi left a comment

Choose a reason for hiding this comment

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

Thanks for spotting the issue and getting a fix out quickly!

@jpelgrom
Copy link
Member Author

jpelgrom commented May 1, 2022

Well it's been almost 1.5 months since the dependency upgrade that causes the issue was merged so I wouldn't say the issue was spotted "quickly"...

@JBassett JBassett merged commit f33743d into home-assistant:master May 1, 2022
@jpelgrom jpelgrom deleted the fix-2477 branch May 1, 2022 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants