DaanHoogland commented on PR #9031:
URL: https://github.com/apache/cloudstack/pull/9031#issuecomment-2092405963
@bernardodemarco , I think your change makes total sense functionally. One
technical concern though
```
public enum Kind {
CSV, Order, Select, WhitespaceSeparatedListWithOptions
}
```
As there is already a `CSV` kind of configuration, do we need to add the
`WhitespaceSeparatedListWithOptions` ? It seems to me it adds no functional
extras in comparison. Isn't converting the `..SecurityProtocol` settings to
`Kind.CSV` good enough?
--
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]
