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

[linky] Resurrect the binding #16586

Merged
merged 4 commits into from Mar 28, 2024
Merged

[linky] Resurrect the binding #16586

merged 4 commits into from Mar 28, 2024

Conversation

clinique
Copy link
Contributor

@clinique clinique commented Mar 27, 2024

The entry point for the authentication and further data requests changed.
Bumped jsoup dependency
Resolves #16585

Signed-off-by: gael@lhopital.org <gael@lhopital.org>
@clinique clinique requested a review from lolodomo as a code owner March 27, 2024 17:43
@clinique clinique self-assigned this Mar 27, 2024
@clinique clinique added the bug An unexpected problem or unintended behavior of an add-on label Mar 27, 2024
Signed-off-by: gael@lhopital.org <gael@lhopital.org>
@clinique
Copy link
Contributor Author

@lolodomo : this is ready for your review.

@lolodomo
Copy link
Contributor

I compile and test that today.

@jlaur
Copy link
Contributor

jlaur commented Mar 27, 2024

@clinique, @lolodomo - it seems like this should probably be cherry-picked to 4.1.x? In that case, do you think it would make sense to extract the library upgrade to another PR to reduce risk for next hotfix?

@lolodomo
Copy link
Contributor

The bundle does not start due to the change of version of org.jsoup:

org.osgi.framework.BundleException: Could not resolve module: org.openhab.binding.linky [293]
  Unresolved requirement: Import-Package: org.jsoup; version="[1.15.0,2.0.0)"

        at org.eclipse.osgi.container.Module.start(Module.java:463) ~[org.eclipse.osgi-3.18.0.jar:?]
        at org.eclipse.osgi.internal.framework.EquinoxBundle.start(EquinoxBundle.java:445) ~[org.eclipse.osgi-3.18.0.jar:?]
        at org.apache.felix.fileinstall.internal.DirectoryWatcher.startBundle(DirectoryWatcher.java:1260) [bundleFile:3.7.4]
        at org.apache.felix.fileinstall.internal.DirectoryWatcher.startBundles(DirectoryWatcher.java:1233) [bundleFile:3.7.4]
        at org.apache.felix.fileinstall.internal.DirectoryWatcher.doProcess(DirectoryWatcher.java:520) [bundleFile:3.7.4]
        at org.apache.felix.fileinstall.internal.DirectoryWatcher.process(DirectoryWatcher.java:365) [bundleFile:3.7.4]
        at org.apache.felix.fileinstall.internal.DirectoryWatcher.run(DirectoryWatcher.java:316) [bundleFile:3.7.4]

Is it possible to avoid this change of version ?

@clinique
Copy link
Contributor Author

clinique commented Mar 27, 2024

The bundle does not start due to the change of version of org.jsoup:

org.osgi.framework.BundleException: Could not resolve module: org.openhab.binding.linky [293]
  Unresolved requirement: Import-Package: org.jsoup; version="[1.15.0,2.0.0)"

        at org.eclipse.osgi.container.Module.start(Module.java:463) ~[org.eclipse.osgi-3.18.0.jar:?]
        at org.eclipse.osgi.internal.framework.EquinoxBundle.start(EquinoxBundle.java:445) ~[org.eclipse.osgi-3.18.0.jar:?]
        at org.apache.felix.fileinstall.internal.DirectoryWatcher.startBundle(DirectoryWatcher.java:1260) [bundleFile:3.7.4]
        at org.apache.felix.fileinstall.internal.DirectoryWatcher.startBundles(DirectoryWatcher.java:1233) [bundleFile:3.7.4]
        at org.apache.felix.fileinstall.internal.DirectoryWatcher.doProcess(DirectoryWatcher.java:520) [bundleFile:3.7.4]
        at org.apache.felix.fileinstall.internal.DirectoryWatcher.process(DirectoryWatcher.java:365) [bundleFile:3.7.4]
        at org.apache.felix.fileinstall.internal.DirectoryWatcher.run(DirectoryWatcher.java:316) [bundleFile:3.7.4]

Is it possible to avoid this change of version ?

Some other bindings are using the same version (Verisure IIRW) - it should work and works in the dev environment. On top of that previous version is signaled with a security issue, reason why I thought it would be a good move. I'll do as you prefer

@lolodomo
Copy link
Contributor

I just put the built jar in my addons folder. I am running a recent snapshot, number 4004.

@jlaur
Copy link
Contributor

jlaur commented Mar 27, 2024

I just put the built jar in my addons folder. I am running a recent snapshot, number 4004.

Does it work if you install the SMGW binding, for example (to fulfill the dependency)?

Signed-off-by: gael@lhopital.org <gael@lhopital.org>
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/linky-communication-error/132889/41

Signed-off-by: gael@lhopital.org <gael@lhopital.org>
@clinique
Copy link
Contributor Author

@lolodomo : you're ready to go, I reverted json dependency to the original version.

Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

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

LGTM, thank you

@lolodomo lolodomo merged commit 9f59d29 into openhab:main Mar 28, 2024
3 checks passed
@lolodomo lolodomo added this to the 4.2 milestone Mar 28, 2024
@lolodomo lolodomo changed the title [linky] Binding stopped working [linky] Resurrect the binding Mar 28, 2024
@lolodomo
Copy link
Contributor

Thank you a lot Gael, I will test it with next snapshot tomorrow.

@lolodomo
Copy link
Contributor

And I will cherry pick it in 4.1.x

lolodomo pushed a commit that referenced this pull request Mar 29, 2024
* Change lincs entrypoint

Signed-off-by: gael@lhopital.org <gael@lhopital.org>
@lolodomo lolodomo added the patch A PR that has been cherry-picked to a patch release branch label Mar 29, 2024
@lolodomo
Copy link
Contributor

I confirm it works. Bravo Gael.

@clinique
Copy link
Contributor Author

I confirm it works. Bravo Gael.

Merci Laurent !

@clinique clinique deleted the linky_16585 branch March 29, 2024 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on patch A PR that has been cherry-picked to a patch release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[linky] Binding stopped working since February
4 participants