Zhengcy05 commented on issue #8182: URL: https://github.com/apache/incubator-seata/issues/8182#issuecomment-5086992715
I took a look at this issue, and my current understanding is that this is a real bug in the server-side local IP selection path. `Server` reads `registry.preferredNetworks` and `registry.ignoredInterfaces` through `ConfigurationFactory.getInstance()`. When `seata.config.type=nacos` is configured, that instance represents the remote config center, so the local Spring Boot `application.yml` value under `seata.registry.*` is skipped. The environment variable still works because Seata checks system/env config first and maps `registry.preferredNetworks` to `REGISTRY_PREFERREDNETWORKS`. A small fix should be to read these registry-local network selection options from `ConfigurationFactory.CURRENT_FILE_INSTANCE`, consistent with how registry configuration is loaded elsewhere, and add regression coverage for the Spring Boot YAML keys. If this direction looks good, I’d like to work on this and submit a PR. -- 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]
