[GitHub] [incubator-druid] FaxianZhao commented on issue #8658: [Lookup] LookupExtractorFactoryMapContainer will cause jdbc password leak issue

[GitBox <gi...@apache.org>]

FaxianZhao commented on issue #8658: [Lookup] LookupExtractorFactoryMapContainer will cause jdbc password leak issue
URL: https://github.com/apache/incubator-druid/issues/8658#issuecomment-564343178
 
 
   > @FaxianZhao Are you using a `DefaultPasswordProvider`? If so, even if the Coordinator used `LookupExtractorFactoryContainer` instead of `LookupExtractorFactoryMapContainer`, the `DefaultPasswordProvider` would still have a visible password in its serialized form.
   > 
   > You could consider using an `EnvironmentPasswordProvider` in your JDBC extraction namespace instead, so the password doesn't appear
   
   Thanks for your help. If we use `LookupExtractorFactoryContainer` in coordinator, we should load all LookupExtractorFactory extensions when it start. Otherwise, coordinator cannot recognize the right implement.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org