From: Gaël L'hopital <gael@lhopital.org>
Date: Sun, 13 Oct 2024 09:50:27 +0000 (+0200)
Subject: [netatmo] Ensure expiresAt is usable (#17553)
X-Git-Url: https://git.basschouten.com/?a=commitdiff_plain;h=1e04957cef6c6abdf76b0c77abb2f1ea6f80fe56;p=openhab-addons.git

[netatmo] Ensure expiresAt is usable (#17553)

* Ensure expiresAt is usable

Signed-off-by: Gaël L'hopital <gael@lhopital.org>
---

diff --git a/bundles/org.openhab.binding.netatmo/src/main/java/org/openhab/binding/netatmo/internal/api/dto/HomeEvent.java b/bundles/org.openhab.binding.netatmo/src/main/java/org/openhab/binding/netatmo/internal/api/dto/HomeEvent.java
index 58b3607a0c..0ec894b756 100644
--- a/bundles/org.openhab.binding.netatmo/src/main/java/org/openhab/binding/netatmo/internal/api/dto/HomeEvent.java
+++ b/bundles/org.openhab.binding.netatmo/src/main/java/org/openhab/binding/netatmo/internal/api/dto/HomeEvent.java
@@ -37,10 +37,15 @@ public class HomeEvent extends Event {
     public class NAEventsDataResponse extends ApiResponse<BodyResponse<Home>> {
     }
 
-    private record Snapshot(String url, ZonedDateTime expiresAt) {
-        // If the snapshot is expired we consider it as not available, so do not provide the url
+    private record Snapshot(@Nullable String url, @Nullable ZonedDateTime expiresAt) {
         public @Nullable String url() {
-            return expiresAt.isAfter(ZonedDateTime.now().withZoneSameInstant(expiresAt.getZone())) ? url : null;
+            ZonedDateTime expires = expiresAt;
+            // If no expiration data provided, lets consider it is available
+            if (expires == null) {
+                return url;
+            }
+            // If the snapshot is expired we consider it as not available, so do not provide the url
+            return expires.isAfter(ZonedDateTime.now().withZoneSameInstant(expires.getZone())) ? url : null;
         }
     }