LI123456mo commented on issue #16432: URL: https://github.com/apache/dubbo/issues/16432#issuecomment-5450573834
> Hi maintainers, I'd like to work on this issue. > > Before implementing the fix, could you confirm that JavaBeanSerializeUtil is expected to preserve a legal null key in Map inputs? > > I plan to first add round-trip regression tests covering null map keys and values, then make the narrowest change possible without weakening the JavaBeanDescriptor validation for regular JavaBean properties. > > If null map keys are intentionally unsupported, I can instead make the validation and error behavior explicit at the map serialization boundary. Yeah they need to confirm if null keys are intentionally unsupported, good thought -- 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]
