Fixed bug that configuration of other processes were not updated when…

… using `nacos grpc client` (#6110)
hyperf · Aug 31, 2023 · 3425f00 · 3425f00
1 parent f7297f0
commit 3425f00
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/CHANGELOG-3.0.md b/CHANGELOG-3.0.md
@@ -4,6 +4,7 @@
 
 - [#6097](https://github.com/hyperf/hyperf/pull/6097) Fixed error that using non-zipkin driver of tracer.
 - [#6099](https://github.com/hyperf/hyperf/pull/6099) Fixed bug that `ConstantFrequency` cannot work when using `redis`.
+- [#6110](https://github.com/hyperf/hyperf/pull/6110) Fixed bug that configuration of other processes were not updated when using `nacos grpc client`.
 
 ## Added
 

diff --git a/src/config-nacos/src/NacosDriver.php b/src/config-nacos/src/NacosDriver.php
@@ -50,9 +50,15 @@ public function createMessageFetcherLoop(): void
 
             $client = $application->grpc->get($tenant);
             $client->listenConfig($group, $dataId, new ConfigChangeNotifyRequestHandler(function (ConfigQueryResponse $response) use ($key, $type) {
-                $this->updateConfig([
-                    $key => $this->client->decode($response->getContent(), $type),
-                ]);
+                $config = $this->client->decode($response->getContent(), $type);
+                $prevConfig = $this->config->get($key, []);
+
+                if ($config !== $prevConfig) {
+                    $this->syncConfig(
+                        [$key => $config],
+                        [$key => $prevConfig],
+                    );
+                }
             }));
         }