Re: Application-accesible Executors

2020-09-22 Thread Romain Manni-Bucau
Le mar. 22 sept. 2020 à 08:54, Martin Grigorov a écrit : > Hi Chris, > > On Fri, Sep 18, 2020 at 7:10 PM Christopher Schultz < > ch...@christopherschultz.net> wrote: > > > All, > > > > I've recently been thinking about application uses of servlet-async and > > Websocket for long-running

Re: WsFilter is missing destroy method

2020-09-21 Thread Romain Manni-Bucau
Hi Ralph, Did you check your api jar was matching tomcat impl version? destroy is in the interface (default method): https://github.com/apache/tomcat/blob/9.0.x/java/javax/servlet/Filter.java#L119 So without more details it sounds you have a dependency conflict or outdated import. Romain Manni

Re: Application-accesible Executors

2020-09-18 Thread Romain Manni-Bucau
, then it is just a matter of defining it as resources in tomcat and do a lookup in any init method to get it I think, we can. Code is trivially extractable from tomee if it is what you have in mind and Apache Geronimo can be a "shared" home for such lib by "design". Hope it makes sense.

Re: [tomcat] branch master updated: Simpler way to determine Graal runtime

2020-07-23 Thread Romain Manni-Bucau
nually). Do I miss something? Le jeu. 23 juil. 2020 à 03:02, Filip Hanik a écrit : > Hi Romain, > > > -Original Message- > > From: Romain Manni-Bucau > > Sent: Wednesday, July 22, 2020 12:48 PM > > To: Tomcat Developers List > > Subject: Re: [

Re: [tomcat] branch master updated: Simpler way to determine Graal runtime

2020-07-22 Thread Romain Manni-Bucau
Thinking out loud: cant you substitute it to be hardcoded to true in native mode? This way you get the best of both. Le mer. 22 juil. 2020 à 20:17, Filip Hanik a écrit : > Thanks Remy, > > > > I ran into some failures when running the test suite using the substrate > VM, but it makes more sense

Re: Jakarta EE APIs

2020-07-22 Thread Romain Manni-Bucau
Le mer. 22 juil. 2020 à 18:29, Mark Thomas a écrit : > On 22/07/2020 17:11, Romain Manni-Bucau wrote: > > Hi Mark, > > > > Another option is to use Apache Geronimo specs (and update/create > > missing ones - think new mail one is not yet there for ex). > > This i

Re: Jakarta EE APIs

2020-07-22 Thread Romain Manni-Bucau
cleans up the legal work for Tomcat as a small side bonus. 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 &

Re: JAX-RPC and Tomcat 10

2020-07-21 Thread Romain Manni-Bucau
Yes, was thinking to tomee in particular since it does not use tomcat as a lib but really as the container so if the container fails then it can become hard if not "disabl-able" somehow (at least with subclassing or something programmatic). Romain Manni-Bucau @rmannibucau <https:

Re: JAX-RPC and Tomcat 10

2020-07-21 Thread Romain Manni-Bucau
Hi Mark, e) c as default + add a toggle to behave as a? (thinking to container extending tomcat where this shouldn't fail probably) Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpres

Re: Native Image - Reflectionless Concept

2020-07-20 Thread Romain Manni-Bucau
:47 AM, Romain Manni-Bucau wrote: > > > > Le lun. 20 juil. 2020 à 17:41, Filip Hanik a écrit : > >> Thanks for chiming in: >> On 7/16/20 6:46 AM, Romain Manni-Bucau wrote: >> >> Hi everyone, >> >> I think the generation is the sanest option since c

Re: Native Image - Reflectionless Concept

2020-07-20 Thread Romain Manni-Bucau
Le lun. 20 juil. 2020 à 17:41, Filip Hanik a écrit : > Thanks for chiming in: > On 7/16/20 6:46 AM, Romain Manni-Bucau wrote: > > Hi everyone, > > I think the generation is the sanest option since code stay clean but it > shouldn't be done in tomcat IMHO but in user code an

Re: Native Image - Reflectionless Concept

2020-07-16 Thread Romain Manni-Bucau
em and use plain java instead (with a good naming convention, it can be generated as well). Keeps the duplication but at least the main code stays clean and optimizations stays together. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net

Re: Support for LetsEncrypt certs, and update process, in Tomcat without restart.

2020-07-14 Thread Romain Manni-Bucau
Le mar. 14 juil. 2020 à 15:50, Merlin Beedell a écrit : > Thank you for the responses. > I can confirm that changing the certificate by replacing the file(s) with > the ones with the same name & password but with an updated certificate > inside does indeed work. The reason I thought otherwise

Re: [ANN] New committer: Raymond Augé

2020-07-02 Thread Romain Manni-Bucau
Congrats Ray, well deserved! 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

Re: Changing the name of the default branch in our git repos

2020-06-16 Thread Romain Manni-Bucau
erior thought people can have as with master or main and it sounds more modern than trunk ;). Indeed, just my 2 cents ;). Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> |

Re: Support for LetsEncrypt certs, and update process, in Tomcat without restart.

2020-06-11 Thread Romain Manni-Bucau
This one was more intended to System.exit but it got aligned with mw impl so it is quite close. Le jeu. 11 juin 2020 à 19:40, Christopher Schultz < ch...@christopherschultz.net> a écrit : > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA256 > > Romain, > > On 6/11/20 1

Re: Support for LetsEncrypt certs, and update process, in Tomcat without restart.

2020-06-11 Thread Romain Manni-Bucau
not. 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 <htt

Re: Support for LetsEncrypt certs, and update process, in Tomcat without restart.

2020-06-08 Thread Romain Manni-Bucau
/meecrowave-letsencrypt/src/main/java/org/apache/meecrowave/letencrypt/LetsEncryptReloadLifecycle.java#L155 for an impl. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github &

Re: API Change - Connector.java Constructor

2020-04-16 Thread Romain Manni-Bucau
Doesn't the same applies? It is a finite product (and not too big). The only choice to do is how to expose it I guess. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com&g

Re: API Change - Connector.java Constructor

2020-04-16 Thread Romain Manni-Bucau
hen you can be fully reflection free using the generated classes instead of the standard one. Hope it makes sense and it is not too late. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannib

Re: [tomcat-jakartaee-migration] branch master updated: Add javax.(decorator|enterprise|inject) as ones which should be migrated

2020-03-13 Thread Romain Manni-Bucau
Hi everyone, Shouldnt tomcat tool stay aligned on tomcat stack? Maven shade or gradle fatjar plugins solve this issue with relocation for all users and all namespaces so not a big deal to not handle more than tomcat IMO, otherwise all hybrid cases (between ~servlet and ee) will be broken IMHO.

Re: Enabling http to https redirects for tomcat.apache.org

2020-02-25 Thread Romain Manni-Bucau
+1 with some light (1 month?) notice time in case anyone uses http directly intentionally, will avoid some security breaches http can get, in particular on subdomains. Le mar. 25 févr. 2020 à 21:45, Christopher Schultz < ch...@christopherschultz.net> a écrit : > -BEGIN PGP SIGNED

Re: [VOTE] Release Apache Tomcat 10.0.0.0-M1

2020-02-06 Thread Romain Manni-Bucau
+1 (non binding) 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

Re: javax -> jakarta rename

2019-12-25 Thread Romain Manni-Bucau
I see a lot of value to the runtime solution cause it is the only one enabling an ops only migration so for all the softwares in "run only" phase it is the only option. However the agent or transformer is the least invasive solution cause it does not require to patch core libs and will avoid app

Re: javax -> jakarta rename

2019-12-21 Thread Romain Manni-Bucau
Hi Adam, Did you evaluate a class transformer added to tomcat classloader? This can stay quite light and enables the same while not using app loader - standard tomcat mode. For app loader case integrations can do the work easily (spring and friends) since they all have asm or equivalent. Hope it

Re: [VOTE] Release Apache Tomcat 9.0.30

2019-12-09 Thread Romain Manni-Bucau
+1 (non binding) Le lun. 9 déc. 2019 à 16:18, Coty Sutherland a écrit : > On Sat, Dec 7, 2019 at 12:24 PM Mark Thomas wrote: > >> The proposed Apache Tomcat 9.0.30 release is now available for voting. >> >> The major changes compared to the 9.0.29 release are: >> >> - Correct multiple

Re: Embedded and default MIME types

2019-12-04 Thread Romain Manni-Bucau
+1 to align both, really reduces diff in tests and prod Le mer. 4 déc. 2019 à 19:23, Mark Thomas a écrit : > On 04/12/2019 18:11, Christopher Schultz wrote: > > Mark, > > > > On 12/4/19 13:02, Mark Thomas wrote: > >> Hi all, > > > >> I was looking at this as an off-shoot of looking at BZ 63985.

Re: RewriteMap parsing

2019-11-01 Thread Romain Manni-Bucau
. 1 nov. 2019 à 18:31, Felix Schumacher < felix.schumac...@internetallee.de> a écrit : > > Am 01.11.19 um 14:24 schrieb Romain Manni-Bucau: > > > > Le ven. 1 nov. 2019 à 11:26, Felix Schumacher < > felix.schumac...@internetallee.de> a écrit : > >> >>

Re: RewriteMap parsing

2019-11-01 Thread Romain Manni-Bucau
Le ven. 1 nov. 2019 à 11:26, Felix Schumacher < felix.schumac...@internetallee.de> a écrit : > > Am 01.11.19 um 11:11 schrieb Romain Manni-Bucau: > > Through the spi IMHO and if it can be ambiguous use an ordinal or priority > to let it be overriden maybe? > >

Re: RewriteMap parsing

2019-11-01 Thread Romain Manni-Bucau
Through the spi IMHO and if it can be ambiguous use an ordinal or priority to let it be overriden maybe? Le ven. 1 nov. 2019 à 10:46, Felix Schumacher < felix.schumac...@internetallee.de> a écrit : > > Am 28.10.19 um 23:06 schrieb Romain Manni-Bucau: > > +1 for quotes >

Re: RewriteMap parsing

2019-10-28 Thread Romain Manni-Bucau
+1 for quotes Can the "function" support be pluggable either with an explicit registry or a SPI? Would be awesome to enrich it in "super tomcat" instances (thinking to meecrowave, tomee and maybe spring boot). Le lun. 28 oct. 2019 à 21:43, Mark Thomas a écrit : > > > On 27/10/2019 11:27, Felix

Re: [VOTE] Private branches in the official Tomcat git repository

2019-10-12 Thread Romain Manni-Bucau
Le sam. 12 oct. 2019 à 15:57, Konstantin Kolinko a écrit : > пт, 11 окт. 2019 г. в 17:21, Rémy Maucherat : > > > > Hi, > > > > This vote is to regulate the use of branches in the official Tomcat > repository beyond branches that are approved by the community such as 8.5.x > and 7.0.x. It is

Re: [VOTE] Private branches in the official Tomcat git repository

2019-10-12 Thread Romain Manni-Bucau
[No] (non binding). As a big github user, i expect main repo to not have unofficially supported code. I work with this kind of setup (for CI) and it is a mess at all layers compared to natural PR structure IMHO. Le sam. 12 oct. 2019 à 02:05, Chuck Caldarale a écrit : > On Oct 11, 2019, at

Re: PooledConnection#connectUsingDriver, Thread.currentThread().getContextClassLoader() is null

2019-08-02 Thread Romain Manni-Bucau
Finally managed to reproduce it in a test: https://github.com/apache/tomcat/pull/183 Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rm

Re: PooledConnection#connectUsingDriver, Thread.currentThread().getContextClassLoader() is null

2019-07-30 Thread Romain Manni-Bucau
Yes, got caught by PooledConnection.class.getClassLoader() which tries to load the driver from tomcat/lib first (common.loader actually) so if you put your driver there it will work in your case. Stays the bug when the driver is only in the webapp. Romain Manni-Bucau @rmannibucau <ht

Re: PooledConnection#connectUsingDriver, Thread.currentThread().getContextClassLoader() is null

2019-07-29 Thread Romain Manni-Bucau
t easily extractable/reproducible > > Maybe : > > th = new Thread() { > @Override > > public void run() { > « do something that requires a connection from the pool » > > } > } ; > th.set getContextClassLoader( null ) ; > th.run() ; > > > > *Von:* Ro

Re: PooledConnection#connectUsingDriver, Thread.currentThread().getContextClassLoader() is null

2019-07-25 Thread Romain Manni-Bucau
app) IMHO. This would also fix your NPE transitively ;). > ------ > Von: Romain Manni-Bucau > Gesendet: Donnerstag, 25. Juli 2019 07:30 > An: Tomcat Developers List > Betreff: Re: PooledConnection#connectUsingDriver, > Thre

Re: PooledConnection#connectUsingDriver, Thread.currentThread().getContextClassLoader() is null

2019-07-24 Thread Romain Manni-Bucau
+1, there is no real other option AFAIK until you init at bootstrap the pool (initial size) to ensure it is in one tomcat classloader. Le jeu. 25 juil. 2019 à 07:08, Clemens Wyss DEV a écrit : > I tried posting this in the tomcat-users-ml, but I guess it rather fits > here: > > Context: >

Re: CDI support improvements

2019-06-17 Thread Romain Manni-Bucau
Le lun. 17 juin 2019 à 16:17, Rémy Maucherat a écrit : > On Mon, Jun 17, 2019 at 10:52 AM Romain Manni-Bucau > wrote: > >> Hi Rémy, >> >> Great progression! Congrats! >> >> I have a few (details) notes - guess i'm opening an open door but just to >>

Re: CDI support improvements

2019-06-17 Thread Romain Manni-Bucau
PS/FYI: just noticed jetty did the same kind of work with weld 4 years ago: https://github.com/eclipse/jetty.project/blob/jetty-9.4.x/jetty-cdi/cdi-servlet/src/main/config/etc/jetty-cdi.xml, great Tomcat is catching up :). Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> |

Re: CDI support improvements

2019-06-17 Thread Romain Manni-Bucau
have a spring config (cxf.xml), not sure it is intended? 5. jettison is not needed since it is not the one used for json serialization (guess you can drop several cxf deps) Hope it makes sense. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibu

Re: CDI support improvements

2019-06-13 Thread Romain Manni-Bucau
@Provider //jaxrs import @Dependent // cdi import public class MyJson extends JsonbJaxrsProvider {} Should be enough if it is the issue (or use delegation instead of inheritance) Le jeu. 13 juin 2019 à 18:45, Rémy Maucherat a écrit : > On Thu, Jun 13, 2019 at 5:31 PM Romain Manni-Bucau > wro

Re: CDI support improvements

2019-06-13 Thread Romain Manni-Bucau
can you drop johnzon-jsonb (+-core + -mapper + jsonp and jsonb specs jars) too? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibuc

Re: CDI support improvements

2019-06-12 Thread Romain Manni-Bucau
Current owb-tomcat module is stable and could be used if some people need it but it does not prevent us to make trunk 9.last based IMHO. Maybe give a shout to owb dev list when you have something you want to integrate and Ill be happy to review and support it. Le mer. 12 juin 2019 à 18:13, Mark

Re: CDI support improvements

2019-06-12 Thread Romain Manni-Bucau
Hell Rémy, I commented inline Le mar. 11 juin 2019 à 23:29, Rémy Maucherat a écrit : > On Tue, Jun 11, 2019 at 9:49 PM Romain Manni-Bucau > wrote: > >> My 2cts would be that we have the luck to be fully ASF here so each >> project can likely get back its missing piece(s

Re: CDI support improvements

2019-06-11 Thread Romain Manni-Bucau
and enable it to align even more on Tomcat in terms of user facing configuration for instance. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com

Re: CDI support improvements

2019-06-11 Thread Romain Manni-Bucau
Le mar. 11 juin 2019 à 16:57, Rémy Maucherat a écrit : > On Thu, May 30, 2019 at 9:35 AM Romain Manni-Bucau > wrote: > > >> Once done it can be hosted on both side.Owb has the advantage to be know >> by users, tomcat to be a more natural home for an integration. At

Re: CDI support improvements

2019-05-30 Thread Romain Manni-Bucau
Hello Rémy, Few precisions inline Le jeu. 30 mai 2019 à 00:34, Rémy Maucherat a écrit : > Hi, > > On Wed, May 29, 2019 at 11:35 PM Romain Manni-Bucau > wrote: > >> Hi Rémy, >> >> Openwebbeans has a tomcat integration module - mainly standalone case, >&g

Re: CDI support improvements

2019-05-29 Thread Romain Manni-Bucau
Hi Rémy, Openwebbeans has a tomcat integration module - mainly standalone case, and meecrowave subproject - embedded or ready to run fatjar. It looks like it covers what you target. Where I am loosing track is why not improving openwebbeans and forking the code in tomcat? At least i would expect

Re: Registry instance aligned on Tomcat/Server lifecycle?

2019-05-15 Thread Romain Manni-Bucau
current issue with the disable method too. Does it sound better? 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 &

Registry instance aligned on Tomcat/Server lifecycle?

2019-05-15 Thread Romain Manni-Bucau
without issues if multiple tomcat instances are started in the same JVM. wdyt? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibuc

Re: Jakarta package change

2019-05-07 Thread Romain Manni-Bucau
I agree with the 2030 statement but it also means not replacing javax by jakarta is saner because it avoids to duplicate and make ambiguous the API and also reduce maintenance cost. It also keeps tomcat embedded working - classloader hacks or javaagent can be tricky to keep performing well with

Re: Becoming graalvm friendly?

2019-04-26 Thread Romain Manni-Bucau
: https://github.com/rmannibucau/docosh/blob/broken-graal-threadlocal/src/main/java/com/github/rmannibucau/docker/compose/cli/svm/Feature_Reflections.java#L35 Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://

Re: Becoming graalvm friendly?

2019-04-24 Thread Romain Manni-Bucau
Awesome news Rémy, thanks for sharing! 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

Re: Becoming graalvm friendly?

2019-03-28 Thread Romain Manni-Bucau
application-development/java-ee-8-high-performance> Le jeu. 28 mars 2019 à 16:19, Rémy Maucherat a écrit : > On Thu, Mar 28, 2019 at 3:43 PM Romain Manni-Bucau > wrote: > > > In my case - I tried on meecrowave - i just switched using Log SPI the > impl > > to somethin

Re: Becoming graalvm friendly?

2019-03-28 Thread Romain Manni-Bucau
tors which is never bad IMHO. 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/rmannibu

Re: Becoming graalvm friendly?

2019-03-20 Thread Romain Manni-Bucau
Le mer. 20 mars 2019 à 15:06, Rémy Maucherat a écrit : > On Tue, Mar 19, 2019 at 4:52 PM Romain Manni-Bucau > wrote: > > > Hi Rémy, well there are two topics here: > > > > 1. how to run tomcat without all that JMX stuff - in Meecrowave for > > instance we dea

Re: Becoming graalvm friendly?

2019-03-19 Thread Romain Manni-Bucau
code path - which would also solve the issue. Finally on the url handler part, it is plain useless in native mode so being able to move that URL.setXXX in a static class we would substrate by a noop can be worth it too. Hope it makes some sense. Romain Manni-Bucau @rmannibucau <https:

Re: [VOTE] Release Apache Tomcat 9.0.17

2019-03-18 Thread Romain Manni-Bucau
+1 (non binding), tested in custom apps and meecrowave 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 &

Re: War Maven artifacts deployment

2019-03-17 Thread Romain Manni-Bucau
Le dim. 17 mars 2019 à 19:35, Gaël Lalire a écrit : > > Le 17 mars 2019 à 16:22, Romain Manni-Bucau a > écrit : > > > Le dim. 17 mars 2019 à 14:51, Gaël Lalire a > > écrit : > > > >> > >> Le 17 mars 2019 à 13:21, Romain Manni-Bucau a > >&

Re: War Maven artifacts deployment

2019-03-17 Thread Romain Manni-Bucau
Le dim. 17 mars 2019 à 14:51, Gaël Lalire a écrit : > > Le 17 mars 2019 à 13:21, Romain Manni-Bucau a > écrit : > > > Le dim. 17 mars 2019 à 12:56, Gaël Lalire a > > écrit : > > > >> Hello Romain, > >> > >> I already explained why I do n

Re: War Maven artifacts deployment

2019-03-17 Thread Romain Manni-Bucau
I checked it. > VestigeJar#open will create an InputStream reading from this locked > RandomAccessFile. > Oki but then you are in jar:file mode ;) > Regards, > Gaël Lalire > > Le 17 mars 2019 à 10:17, Romain Manni-Bucau a > écrit : > > Hi Gaël, > > In Tomee we

Re: War Maven artifacts deployment

2019-03-17 Thread Romain Manni-Bucau
Hi Gaël, In Tomee we plugged before to enrich the classloader and then tomcat -and other libs - works normally using jar urls. Cant you use a listener to do that and convert m2 urls to plain jar files - at the end it is local files i guess otherwise you generally consume too much memory to be

Becoming graalvm friendly?

2019-03-10 Thread Romain Manni-Bucau
2. any will to work in that direction in the community? (graalvm is still very young and lack several features to really embrace java ecosystem so can be fair to say "later") Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> |

Re: Proposed change to STREAMS_DROP_EMPTY_MESSAGES system property

2019-02-05 Thread Romain Manni-Bucau
As an user c sounds ok. Worse case maybe keep it for 1 release with some comm to have time to migrate if relevant. Le mar. 5 févr. 2019 à 20:10, Rémy Maucherat a écrit : > On Tue, Feb 5, 2019 at 7:58 PM Mark Thomas wrote: > > > Hi, > > > >

Re: System properties: JVM launch versus catalina.properties

2018-12-30 Thread Romain Manni-Bucau
except the name, if run in the same JVM thantomcat it is too late anyway so 2 options are either to not do it or preprocess it IMHO Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpres

Re: System properties: JVM launch versus catalina.properties

2018-12-28 Thread Romain Manni-Bucau
. 2018 19:41, Christopher Schultz < ch...@christopherschultz.net> a écrit : > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA256 > > Romain, > > On 12/28/18 10:10, Romain Manni-Bucau wrote: > > Hi > > > > As a user it is nicer to be able to set all system pro

Re: System properties: JVM launch versus catalina.properties

2018-12-28 Thread Romain Manni-Bucau
Hi As a user it is nicer to be able to set all system properties in the same place and catalina code is too late for some of them like log manager. Why not having a conf/system.properties handled by scripts and in fallback by Tomcat (embedded case)? Would just be a more natural way to write it

Re: open visibility of JavaClassCacheEntry

2018-12-06 Thread Romain Manni-Bucau
Fair enough, done in https://bz.apache.org/bugzilla/show_bug.cgi?id=62986 Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibuc

Re: open visibility of JavaClassCacheEntry

2018-12-06 Thread Romain Manni-Bucau
Hello Rémy, Le jeu. 6 déc. 2018 à 14:13, Rémy Maucherat a écrit : > On Thu, Dec 6, 2018 at 11:51 AM Romain Manni-Bucau > wrote: > > > Hi guys, > > > > can you make ContextConfig.JavaClassCacheEntry public please? Idea is to > be > > able to override Con

Re: open visibility of JavaClassCacheEntry

2018-12-06 Thread Romain Manni-Bucau
PS: alternative which can be saner would be to split webConfig() in a set of small protected methods instead of a monolitic block, typically here a processAnnotations(webXmls) not depending on internals can work Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

open visibility of JavaClassCacheEntry

2018-12-06 Thread Romain Manni-Bucau
Hi guys, can you make ContextConfig.JavaClassCacheEntry public please? Idea is to be able to override ContextConfig and potentially customize processAnnotations methods. Currently it is a pain and it is preventing to upgrade meecrowave and likely tomee to java11+jlink support Romain Manni-Bucau

Re: Problems running TC 7 with Log4J 1.2 and Java 11

2018-11-22 Thread Romain Manni-Bucau
that either for some reason the server loader is no longer > parent of the common loader or the getResource() call changed behavior > w.r.t. class loader hierarchies and delegation. > > Will debug further. > > Regards, > > Rainer > > Am 22.11.2018 um 18:31 schrieb Romai

Re: Problems running TC 7 with Log4J 1.2 and Java 11

2018-11-22 Thread Romain Manni-Bucau
Hi Rainer, You are right, missed it was set OOTB. BTW, just tested on tomcat 7.0.92 with the log4j (v1.2.17) and its adapter (of the 8.0.53) and java 11.0.1+13-LTS and it works for me (base=home in my test). Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

Re: Problems running TC 7 with Log4J 1.2 and Java 11

2018-11-22 Thread Romain Manni-Bucau
Hi Rainer, did you open some java.base modules? like --add-opens java.base/java.lang=log4j (not sure this is the one to open but I guess you can debug and identified missing open this way - debugging java.lang.Module#isOpen(java.lang.String) for instance) Romain Manni-Bucau @rmannibucau <ht

Re: AccessLogValve using STDOUT

2018-11-12 Thread Romain Manni-Bucau
@Rainer: if it helps you can test with this file handler: https://github.com/apache/tomee/blob/master/tomee/tomee-juli/src/main/java/org/apache/tomee/jul/handler/rotating/LocalFileHandler.java Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

Re: AccessLogValve using STDOUT

2018-11-11 Thread Romain Manni-Bucau
Le dim. 11 nov. 2018 à 15:41, Christopher Schultz < ch...@christopherschultz.net> a écrit : > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA256 > > Romain, > > On 11/11/18 05:57, Romain Manni-Bucau wrote: > > I think the perf point is likely wrong. Defa

Re: AccessLogValve using STDOUT

2018-11-11 Thread Romain Manni-Bucau
just a jar to add implementing Log SPI). 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.linke

Re: AccessLogValve using STDOUT

2018-11-11 Thread Romain Manni-Bucau
Hi Rainer, There is an abstract access valve do providing a log impl (like [1]) can enable that - plus other standard stuff like pushing on kafka accesses - without hardcoding an stdout stream which can not work in docker in some setup (where tomcat is launched by another process and redirects

Re: [VOTE] Release Apache Tomcat 9.0.13

2018-11-04 Thread Romain Manni-Bucau
+1 (non-binding), tested on meecrowave and some work projects 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 &

Re: Don't require a base directory?

2018-07-19 Thread Romain Manni-Bucau
up? :) 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> |

Don't require a base directory?

2018-07-15 Thread Romain Manni-Bucau
? 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 <htt

Re: Dynamic reloading of SSL certificates

2018-06-27 Thread Romain Manni-Bucau
+1 for connectors IMHO Le mer. 27 juin 2018 18:21, Christopher Schultz < ch...@christopherschultz.net> a écrit : > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA256 > > Romain, > > On 6/27/18 11:50 AM, Romain Manni-Bucau wrote: > > up? any hope we have live reloadi

Re: Dynamic reloading of SSL certificates

2018-06-27 Thread Romain Manni-Bucau
up? any hope we have live reloading of certs in tomcat? 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 &

Re: [VOTE] Release Apache Tomcat 9.0.10

2018-06-21 Thread Romain Manni-Bucau
+1 (non-binding), tested on Meecrowave and a few work apps and all was green Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibuc

Re: Tomcat 9 Support Java Version clarification

2018-04-26 Thread Romain Manni-Bucau
@Chris: -release 1.8 (kind of target=source=1.8 but java 9 has the signatures of java 8 to ensure it will run on java 8 and avoid the issues like "i built with java 8 with source=target=1.7 but it doesnt run on java 7") Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannib

Re: [Git migration] trunk or master

2018-01-09 Thread Romain Manni-Bucau
+1 for master and same as on svn for branches Le 10 janv. 2018 00:03, "Emmanuel Bourg" a écrit : > Le 08/01/2018 à 15:33, Mark Thomas a écrit : > > > Thoughts? > > +1 for master > > - > To unsubscribe,

Re: Dynamic reloading of SSL certificates

2018-01-02 Thread Romain Manni-Bucau
Yes, if tomcat can supports hot reloading of certs it is very feasible: https://github.com/rmannibucau/letsencrypt-manager/blob/master/src/main/java/com/github/rmannibucau/letsencrypt/manager/LetsEncryptManager.java Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

Re: Dynamic reloading of SSL certificates

2018-01-02 Thread Romain Manni-Bucau
up? 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> 2017-09-0

Re: removeMessageHandler in WS doesnt handle wrapped listeners?

2017-12-21 Thread Romain Manni-Bucau
PS: forgot the workaround: session.removeMessageHandler(session.getMessageHandlers().iterator().next()); Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github &

removeMessageHandler in WS doesnt handle wrapped listeners?

2017-12-21 Thread Romain Manni-Bucau
the instance is wrapped but not tested this way, i would need to remove the handler passing the wrapped instance but i cant access it transparently through the javax code. Did I miss something? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.met

Re: [VOTE] Release Apache Tomcat 9.0.2

2017-11-27 Thread Romain Manni-Bucau
2017-11-27 9:15 GMT+01:00 Konstantin Kolinko <knst.koli...@gmail.com>: > 2017-11-27 10:55 GMT+03:00 Romain Manni-Bucau <rmannibu...@gmail.com>: >> Hi guys, >> >> I have 2 questions: >> >> 1. (simple ;)) is the repo released? central is not yet synch-

Fwd: [VOTE] Release Apache Tomcat 9.0.2

2017-11-26 Thread Romain Manni-Bucau
Hi guys, I have 2 questions: 1. (simple ;)) is the repo released? central is not yet synch-ed 2. "java 9 fully supported": I didn't see the classloader update to support multi-jar release or module-info respect (visibility+SPI), is it intended or is it "support" as "can

tomcat-api missing from websocket?

2017-11-02 Thread Romain Manni-Bucau
Hi guys, is it known the tomcat-websocket pom doesn't contain all dependencies websocket code requires? typically i didnt see tomcat-api here but WsSession uses it. Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | LinkedIn

Re: Java 9, modularisation and build systems

2017-10-06 Thread Romain Manni-Bucau
Hi Mark, few comments inline 2017-10-06 10:18 GMT+02:00 Mark Thomas : > Hi all, > > As you have probably seen, I've been working on improving Java 9 > support. The current TODO list is: > > - module path scanning > - handling multi-release JARs in the JarScanner > > I've been

Re: Dynamic reloading of SSL certificates

2017-09-05 Thread Romain Manni-Bucau
Hello guys, wonder if this thread went anywhere? Would be very neat to have a let's encrypt integration (don't know if it would be a listener to declare to have automatic reloading or just a flag on the SSL config but it would ease deploying self hosted instances). Romain Manni-Bucau

symmetry of get/set property in endpoint?

2017-07-06 Thread Romain Manni-Bucau
Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://blog-rmannibucau.rhcloud.com> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | JavaEE

Re: RFE idea: Add port offset system property to Tomcat

2017-06-09 Thread Romain Manni-Bucau
@Mark: guess you are thinking to server.xml? then it would have placeholder support so still a system property somehow no? +1 anyway, sounds very useful! Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://blog-rmannibucau.rhcloud.com> | Ol

Re: Things that we can do to increase contributor involvement?

2017-06-01 Thread Romain Manni-Bucau
t have ant), and make it easier to browse the structure/package/module Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://blog-rmannibucau.rhcloud.com> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com

scanning in embedded mode

2017-05-04 Thread Romain Manni-Bucau
Hi guys, on java 9 in embedded mode the app loader is no more a URLClassLoader so StandardScanner "fails" silently. Is it plan to read java.class.path as a fallback or something else? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://blog-rman

  1   2   3   4   >