Fix update actor failing because of a serializer error (#1098)

MbinOrg · Sep 9, 2024 · 26bc1c8 · 26bc1c8
1 parent 2069352
commit 26bc1c8
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/src/Service/RemoteInstanceManager.php b/src/Service/RemoteInstanceManager.php
@@ -57,11 +57,18 @@ public function updateInstance(Instance $instance, bool $force = false): bool
 
             $nodeInfoRaw = $this->client->fetchInstanceNodeInfo($linkToUse->href, false);
             $this->logger->debug('got raw nodeinfo for url {url}: {raw}', ['raw' => $nodeInfoRaw, 'url' => $linkToUse]);
-            /** @var NodeInfo $nodeInfo */
-            $nodeInfo = $serializer->deserialize($nodeInfoRaw, NodeInfo::class, 'json');
-
-            $instance->software = $nodeInfo?->software?->name;
-            $instance->version = $nodeInfo?->software?->version;
+            try {
+                /** @var NodeInfo $nodeInfo */
+                $nodeInfo = $serializer->deserialize($nodeInfoRaw, NodeInfo::class, 'json');
+                $instance->software = $nodeInfo?->software?->name;
+                $instance->version = $nodeInfo?->software?->version;
+            } catch (\Error|\Exception $e) {
+                $this->logger->warning('There as an exception decoding the nodeinfo from {url}: {e} - {m}', [
+                    'url' => $instance->domain,
+                    'e' => \get_class($e),
+                    'm' => $e->getMessage(),
+                ]);
+            }
             $instance->setUpdatedAt();
             $this->entityManager->persist($instance);