Request to expose certain protected APIs as public about logging-log4j2

[ppkarwasz]

Hello @Karthi1711,

Can You submit a PR with the changes?

[garydgregory]

I think we want to be careful here @ppkarwasz: Do we really want to lock down an API as public for such low level classes and be bound to binary compatibility? Or are we going to say this class is not part of BC?

[ppkarwasz]

@garydgregory,

I would just prefer to see this request in the form of a PR, that we can discuss on the code.

[Karthi1711]

PR created: #1683.

[vy]

@garydgregory, I see the points you raise. We already expect users to implement RolloverStrategys, which is a public interface. In all of our own implementations of RolloverStrategy, we reach out to FileExtension methods the user is requesting to expose. Hence, I find it fair that custom RolloverStrategys should have access to those methods.