bernardodemarco commented on PR #10790: URL: https://github.com/apache/cloudstack/pull/10790#issuecomment-2842595375
@DaanHoogland, I’ve identified a possible inconsistency with configurations that accept lists of values (such as those of type CSV). Their default value is an empty string, so when the `resetConfiguration` API is called, the value is reset to an empty string, as expected. However, when the `updateConfiguration` API is used with the `value` parameter set to an empty string, the Management Server currently stores `NULL` for the configuration in the database. I believe this could lead to bugs down the line, such as potential NPEs. I suggest opening a separate issue and PR to address this behavior, as it's not directly related to the scope of this PR. What do you guys think? (cc. @winterhazel, @JoaoJandre, @sureshanaparti) -- 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]
