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

Upgrade json to 20231013 #15745

Merged
merged 2 commits into from
Oct 27, 2023
Merged

Upgrade json to 20231013 #15745

merged 2 commits into from
Oct 27, 2023

Conversation

jlaur
Copy link
Contributor

@jlaur jlaur commented Oct 12, 2023

Release notes:

Fixed vulnerabilities:

Library:

See also dependabot alerts.

Release notes

Sourced from org.json:json's releases.

20231013

Pull Request Description
#793 Reverted #761
#792 update the docs for release 20231013
#783 optLong vs getLong inconsistencies
#782 Fix XMLTest.testIndentComplicatedJsonObjectWithArrayAndWithConfig() for Windows
#779 add validity check for JSONObject constructors
#778 Fix XMLTest.testIndentComplicatedJsonObjectWithArrayAndWithConfig() for Windows
#776 Update [JUnit to version 4.13.2
#774 Removing unneeded synchronization
#773 Add optJSONArray method to JSONObject with a default value
#772 Disallow nested objects and arrays as keys in objects
#779 Unit test cleanup
#769 Addressed Java 17 compile warnings
#764 Update CodeQL action version
#761 Add module-info
#759 JSON parsing should detect embedded 
#753 Updated new object methods
#752 Fixes possible unit test bug when compiling/testing on Windows

20230618

Pull Request Description
#749 Prep for release 20230618
#740 Fixed Flaky Tests Caused by JSON permutations
#734 Fixed Flaky Tests Caused by JSON permutations
#733 JSONTokener implemented java.io.Closeable
#731 Removing commented out code in JSONObject optDouble()
#729 Refactor ParserConfiguration class hierarchy

Yet untested, since I don't use any of those bindings. I'm hoping binding maintainers can help out.

Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
@jlaur jlaur added security additional testing preferred The change works for the pull request author. A test from someone else is preferred though. labels Oct 12, 2023
Copy link
Contributor

@weymann weymann left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
@jlaur jlaur changed the title Upgrade json to 20230618 Upgrade json to 20231013 Oct 16, 2023
@lolodomo
Copy link
Contributor

@jlaur: openhabcloud is certainly used by almost every user. Any idea what exactly could impact this change or what to test WRT openhabcloud ?

@lolodomo
Copy link
Contributor

@weymann : you approved after testing which one of the impacted bindings ?

@jlaur
Copy link
Contributor Author

jlaur commented Oct 19, 2023

openhabcloud is certainly used by almost every user. Any idea what exactly could impact this change or what to test WRT openhabcloud ?

It's used only by this class it seems, but in many places:

Best would probably be if someone would be willing to install it in a production system and let it run for a while. I don't use the binding personally (I'm using VPN for remote access through the openHAB app and Pushover for notifications).

I wouldn't expect any issues when looking at the release notes, but you never really know. I think the binding would be quite broken in general if the new version is bad.

Copy link
Member

@wborn wborn left a comment

Choose a reason for hiding this comment

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

Thanks and let's give it a test. 🚀

@wborn wborn merged commit a357f7f into openhab:main Oct 27, 2023
3 checks passed
@wborn wborn added this to the 4.1 milestone Oct 27, 2023
@wborn wborn removed the additional testing preferred The change works for the pull request author. A test from someone else is preferred though. label Oct 27, 2023
@jlaur jlaur deleted the json-upgrade branch October 27, 2023 18:54
truidix pushed a commit to truidix/openhab-addons that referenced this pull request Oct 30, 2023
Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants