-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove Undertow dependency from ArC #529
Conversation
import org.jboss.builder.item.SimpleBuildItem; | ||
import org.jboss.protean.arc.ArcContainer; | ||
|
||
public final class ArcContainerBuildItem extends SimpleBuildItem { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, I think we should rather extend the BeanContainer
. Maybe just expose org.jboss.protean.arc.ArcContainer.withinRequest(Supplier<T>)
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have updated this
|
||
public class BeanDefiningAnnotationsBuildStep { | ||
|
||
@BuildStep | ||
void beanDefiningAnnotations(Capabilities capabilities, BuildProducer<BeanDefiningAnnotationBuildItem> annotations) { | ||
if (capabilities.isCapabilityPresent(Capabilities.UNDERTOW)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Capabilities.UNDERTOW
is not used anywhere else and could be removed.
6ed34b0
to
71eb854
Compare
- also do not activate the request context if already active
Make BeanContainer.RequestAction more generic
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
No description provided.