leno23 opened a new pull request, #3417:
URL: https://github.com/apache/dubbo-go/pull/3417

   ## What changed
   
   - Make `ScriptRouter.Process` ignore non-string config-center payloads 
instead of panicking.
   - Add a regression test for non-string config updates to keep the router 
disabled and unchanged.
   
   ## Why
   
   Issue #3203 lists a router cleanup item to fix `ScriptRouter.Process()` 
panicking on failed type assertions. Config-center events should not crash the 
process when they carry an unexpected value type; the router can log and skip 
the invalid update instead.
   
   ## Validation
   
   - `go test ./cluster/router/script`
   - `go test ./cluster/router/...`
   - `git diff --check`
   
   Refs #3203.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to