arunsrajan commented on code in PR #26010:
URL: https://github.com/apache/camel/pull/26010#discussion_r3916112410
##########
components/camel-alibaba/camel-alibaba-common/src/main/java/org/apache/camel/component/alibaba/common/models/ServiceKeys.java:
##########
@@ -16,32 +16,11 @@
*/
package org.apache.camel.component.alibaba.common.models;
-public class ServiceKeys {
-
- private String accessKey;
- private String secretKey;
+public record ServiceKeys(
+ String accessKey,
+ String secretKey) {
Review Comment:
Thanks @gnodet for catching this!
I have reverted `ServiceKeys` back to a standard mutable Java class with
`getAccessKey()`, `setAccessKey(...)`, `getSecretKey()`, and
`setSecretKey(...)`, along with default and all-args constructors.
All helper accessors across `OpenApiClientSupport`, `OSSUtils`, and
`MNSUtils` have been updated to use the getters, ensuring nested property
binding via `PropertyBindingSupport` (e.g.
`camel.component.alibaba-eventbridge.serviceKeys.accessKey=xxx`) and Spring
Boot auto-configuration remain fully compatible across all 8 Alibaba modules.
--
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]
