You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "llowinge (via GitHub)" <gi...@apache.org> on 2025/01/06 13:26:41 UTC

[I] Replace @SessionScoped with @ApplicationScoped in Camel CXF [camel-quarkus]

llowinge opened a new issue, #6888:
URL: https://github.com/apache/camel-quarkus/issues/6888

   ### Bug description
   
   As discussed in https://github.com/apache/camel-quarkus/issues/6880 it looks we are using `@SessionScoped` at more places (https://github.com/search?q=repo%3Aapache%2Fcamel-quarkus%20%40SessionScoped&type=code). We should aim for `@ApplicationScoped` (or `@RequestScope` for special needs, but `@SessionScoped` acted as workaround only - there is no session in the camel cxf route, so it doesn't make sense to use it).


-- 
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: commits-unsubscribe@camel.apache.org.apache.org

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


Re: [I] Replace @SessionScoped with @ApplicationScoped in Camel CXF [camel-quarkus]

Posted by "llowinge (via GitHub)" <gi...@apache.org>.
llowinge commented on issue #6888:
URL: https://github.com/apache/camel-quarkus/issues/6888#issuecomment-2573477495

   First observation, using `@RequestScoped` is giving me 
   ```
   Caused by: jakarta.enterprise.context.ContextNotActiveException: RequestScoped context was not active when trying to obtain a bean instance for a client proxy of PRODUCER_METHOD bean [class=org.apache.camel.quarkus.component.cxf.soap.client.it.CxfSoapClientRoutes, id=XzFNO2ttWexJbzVIRqtS-_USgok]
   	- you can activate the request context for a specific method using the @ActivateRequestContext interceptor binding
   ```


-- 
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: commits-unsubscribe@camel.apache.org

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


Re: [I] Replace @SessionScoped with @ApplicationScoped in Camel CXF [camel-quarkus]

Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton closed issue #6888: Replace @SessionScoped with @ApplicationScoped in Camel CXF
URL: https://github.com/apache/camel-quarkus/issues/6888


-- 
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: commits-unsubscribe@camel.apache.org

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


Re: [I] Replace @SessionScoped with @ApplicationScoped in Camel CXF [camel-quarkus]

Posted by "llowinge (via GitHub)" <gi...@apache.org>.
llowinge commented on issue #6888:
URL: https://github.com/apache/camel-quarkus/issues/6888#issuecomment-2573518173

   @jamesnetherton @ppalaga Would you be OK with sth like https://github.com/llowinge/camel-quarkus/commit/7d75b2240c08d28143e049edc64c5a6874b3db35 ?


-- 
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: commits-unsubscribe@camel.apache.org

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


Re: [I] Replace @SessionScoped with @ApplicationScoped in Camel CXF [camel-quarkus]

Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton commented on issue #6888:
URL: https://github.com/apache/camel-quarkus/issues/6888#issuecomment-2574547929

   > Would you be OK with sth like [llowinge@7d75b22](https://github.com/llowinge/camel-quarkus/commit/7d75b2240c08d28143e049edc64c5a6874b3db35) ?
   
   From my POV, yes. It's the more correct solution.


-- 
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: commits-unsubscribe@camel.apache.org

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


Re: [I] Replace @SessionScoped with @ApplicationScoped in Camel CXF [camel-quarkus]

Posted by "llowinge (via GitHub)" <gi...@apache.org>.
llowinge commented on issue #6888:
URL: https://github.com/apache/camel-quarkus/issues/6888#issuecomment-2573415499

   @jamesnetherton Can you assign me ?


-- 
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: commits-unsubscribe@camel.apache.org

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