turcsanyip commented on PR #10567: URL: https://github.com/apache/nifi/pull/10567#issuecomment-3589648977
> It is extremely unlikely but the dynamic properties are configured with support for expression language at environment level. Users should definitely use parameters instead but they could technically reference environment variable through expression language. Do we want to do something about it? Right now we would not evaluate the dynamic property with expression language for the SASL extensions. Good catch @pvillard31, I did not notice it. Checked `PropertyContext` parameter in `OAuthBearerLoginConfigProvider` (where the dynamic properties are used) and it does not support enumerating (or getting the collection of) `PropertyDescriptiors` / `PropertyValues` that would be needed to check the property type and then evaluate the value. So the viable way is to disable EL support for these properties. -- 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]
