fix(clustering): session renamed fields (#10354)

Co-authored-by: Samuele Illuminati <[email protected]>

CHANGELOG.md

@@ -80,6 +80,8 @@
 
 - Fix an issue where control plane does not downgrade config for `aws_lambda` and `zipkin` for older version of data planes.
   [#10346](https://github.com/Kong/kong/pull/10346)
+- Fix an issue where control plane does not rename fields correctly for `session` for older version of data planes.
+  [#10352](https://github.com/Kong/kong/pull/10352)
 
 ## 3.2.0
 

kong/clustering/compat/init.lua

@@ -238,6 +238,7 @@ end
 local function rename_field(config, name_from, name_to, has_update)
   if config[name_from] ~= nil then
     config[name_to] = config[name_from]
+    config[name_from] = nil
     return true
   end
   return has_update

spec/01-unit/19-hybrid/03-compat_spec.lua

@@ -157,6 +157,9 @@ describe("kong.clustering.compat", function()
             "goodbye",
             "my.nested.field",
           },
+          session = {
+            "anything",
+          },
         },
       })
     end)
@@ -275,6 +278,37 @@ describe("kong.clustering.compat", function()
           },
         },
       },
+
+      {
+        name = "renamed fields",
+        version = "1.0.0",
+        plugins = {
+          {
+            name = "session",
+            config = {
+              idling_timeout = 60,
+              rolling_timeout = 60,
+              stale_ttl = 60,
+              cookie_same_site = "Default",
+              cookie_http_only = false,
+              remember = true,
+            },
+          },
+        },
+        expect = {
+          {
+            name = "session",
+            config = {
+              cookie_idletime = 60,
+              cookie_lifetime = 60,
+              cookie_discard = 60,
+              cookie_samesite = "Lax",
+              cookie_httponly = false,
+              cookie_persistent = true,
+            },
+          },
+        },
+      },
     }
 
     for _, case in ipairs(cases) do