MikeThomsen commented on pull request #5319:
URL: https://github.com/apache/nifi/pull/5319#issuecomment-905661035


   > I would be curious how the previous solution was tested in a real-life 
environment.
   
   It started as a controller service for a private bundle I have been 
maintaining for a client. It's not the only component we have on site that uses 
it. We tend to start testing with password flows first and then move to client 
credentials as we move toward production.
   
   The logic behind the implementation is that the controller service is meant 
to be dumb with regard to the specific credentials and provide functionality to 
a particular process with its own configured credentials.
   
   One major concern I have is with the naming convention. Password* implies a 
particular OAuth2 strategy, but the service supports both password and client 
credential flows.


-- 
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]


Reply via email to