Hi Richard, I still think SystemCurrentTime should have a public noarg constructor (or just drop the private one) since it will enable tomee to fully configure dynamically the tx mgr with this new feature but otherwise +1 to apply them all.
Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book <https://www.packtpub.com/application-development/java-ee-8-high-performance> Le lun. 22 mars 2021 à 17:03, Zowalla, Richard < richard.zowa...@hs-heilbronn.de> a écrit : > Hi all, > > wanted to raise attention on this again. 6792 would be very nice as we > should allow TLS/SSL protocol versions for a given mail server instead > of falling back to some hard-coded default. > > Gruss > Richard > > Am Mittwoch, den 24.02.2021, 09:33 +0100 schrieb Romain Manni-Bucau: > > Hi all, > > > > AFAIK we have a few pending patches to apply/issue to close: > > > > - [mail] https://issues.apache.org/jira/browse/GERONIMO-6792: update > > some defaults and config capacity > > - [mail] https://issues.apache.org/jira/browse/GERONIMO-6801 and > > https://issues.apache.org/jira/browse/GERONIMO-6800 (setText) > > - [transaction-manager] > > https://issues.apache.org/jira/browse/GERONIMO-6805: enable to change > > the time evaluator impl > > > > If someone else can have a review it would be great (feel free to > > apply the patch or I can do it after). > > > > note: some of the patches are waiting for some feedback - in > > particular txmgr one, wonder about tomee <resource> usage which can > > need to remove the private constructor of the default impl to enable > > to configure the impl completely. > > > > Thanks, > > Romain Manni-Bucau > > @rmannibucau | Blog | Old Blog | Github | LinkedIn | Book > >