pvillard31 commented on PR #11604:
URL: https://github.com/apache/nifi/pull/11604#issuecomment-5490671769

   Thanks for the review. I simplified the implementation substantially and 
removed the database-specific validation from the password provider. The 
provider now focuses only on obtaining, scoping, caching, refreshing, and 
returning the Cloud SQL IAM token. JDBC URL, driver, username, and TLS 
configuration remain the responsibility of the DBCP service and JDBC driver.
   
   I also removed the exception stack-trace introspection and reduced the tests 
to the essential credential lifecycle, refresh, verification, and 
failure-handling behavior. The focused tests, complete GCP processor module 
suite, Checkstyle, RAT, and PMD all pass. I also completed final live 
validation with both PostgreSQL and MySQL.


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