vy commented on issue #3074:
URL: 
https://github.com/apache/logging-log4j2/issues/3074#issuecomment-2404633198

   > If, during the retry, the key/truststore are reloaded, then the latest 
certs would always be used in re-establishing the connection and would 
effectively remove the need to trigger the reconfiguration.
   
   @MichaelMorrisEst, this is something crossed my mind while working on #2767, 
but I am not sure about the implications of trying to reload key/trust store on 
every new socket creation attempt. [Thinking out loud here...] If the server 
has indeed gained a new identity, reloading key/trust stores will solve the 
problem swiftly. But what if the failure is due to something else? If we decide 
to implement this, shall this be the default, or opt-in via a new attribute?
   
   Another concern I have regarding this approach is we will be introducing an 
exception to the reconfiguration logic. That is, when a Log4j configuration is 
loaded, it is treated more or less immutable – obviously, except 
`LoggerConfig`s. Now we will introduce a _"please reload only this little 
configuration element"_ exception. I am not saying this is something bad per 
se. But exceptions generally bite us in the long run.
   
   @ppkarwasz, any thoughts?


-- 
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: notifications-unsubscr...@logging.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to