Martin Kouba

> > We had many problems with concurrency access, session leaks, context propagation, etc. > > Removing the functionality does not solve the problem. It merely pushes the problem somewhere...

> Note that the best way I found to do this right now is the following: Where do you close the session? BTW your solution seems to be similar to...

> There's no "problem" with session destruction, as is demonstrated in the code I already posted above. @gavinking So I looked at your _solution_ and it's very different from the...

> Wait wait wait. What if the code were this: > > @AroundInvoke > public Object withSession(InvocationContext invocationContext) throws Exception { > if ( invocationContext.getMethod().getReturnType().equals(Uni.class) ) { > return factory.withStatelessTransaction(session...

> I'm not sure if we should make `Mutiny.Session` available for injection. Perhaps `Uni` is enough? > > In any case, we should probably move `SessionOperations`, `@WithSession`, `@WithTransaction` to Quarkus-HR...

> @mkouba you might be interested in this failure > > ``` > 2025-06-11T15:52:04.8646153Z [ERROR] NonAppEndpointsDisabledTest » Runtime io.quarkus.builder.BuildException: Build failure: Build failed due to errors > 2025-06-11T15:52:04.8647800Z [error]: Build...

> @mkouba would you like this change to be merged into the main branch? I'd like to find the problem first but yes, it might be useful.

> I suspect it might just be a rare glitch Ok, let's keep the assert then...

> Just to clarify, the mock mailbox is a CDI bean: > > ``` > syntheticBeans.produce(SyntheticBeanBuildItem.configure(MockMailbox.class) > .scope(Singleton.class) > .qualifiers(qualifier) > .unremovable() > .defaultBean() > .setRuntimeInit() > .addInjectionPoint(ClassType.create(DotName.createSimple(Mailers.class))) > .createWith(recorder.mockMailboxFunction(name,...