Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add support of jetty 12 ee10/ee9 #128

Merged
merged 2 commits into from
Oct 12, 2022
Merged

Add support of jetty 12 ee10/ee9 #128

merged 2 commits into from
Oct 12, 2022

Conversation

olamy
Copy link
Contributor

@olamy olamy commented Oct 9, 2022 •
edited
Loading

Short description of what this resolves:

Adding support of Jetty 12

Changes proposed in this pull request:

  • add support of Jetty 12 ee10
  • add support of Jetty 12 ee9

@olamy olamy changed the title [WIP] start work on adding jetty 12 ee10 Add support of jetty 12 ee10/ee9 Oct 11, 2022
@olamy olamy marked this pull request as ready for review October 11, 2022 11:29
@olamy olamy requested a review from joakime October 11, 2022 11:29
joakime
joakime reviewed Oct 11, 2022
View reviewed changes
Copy link
Contributor

@joakime joakime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Get rid of the AccessController use in Java 17 minimum projects.

...c/main/java/org/jboss/arquillian/container/jetty/embedded_12_ee10/ArquillianAppProvider.java Outdated Show resolved Hide resolved
.../main/java/org/jboss/arquillian/container/jetty/embedded_12_ee10/JettyEmbeddedContainer.java Show resolved Hide resolved
.../main/java/org/jboss/arquillian/container/jetty/embedded_12_ee10/JettyEmbeddedContainer.java Show resolved Hide resolved
...rc/main/java/org/jboss/arquillian/container/jetty/embedded_12_ee9/ArquillianAppProvider.java Outdated Show resolved Hide resolved
...c/main/java/org/jboss/arquillian/container/jetty/embedded_12_ee9/JettyEmbeddedContainer.java Show resolved Hide resolved
...c/main/java/org/jboss/arquillian/container/jetty/embedded_12_ee9/JettyEmbeddedContainer.java Show resolved Hide resolved
@olamy olamy merged commit 3c7b0f4 into master Oct 12, 2022
@olamy olamy deleted the jetty-12 branch October 12, 2022 00:28
@olamy
Copy link
Contributor Author

olamy commented Dec 2, 2022

@starksm64 could we get a release with this change? thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joakime joakime joakime approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@olamy @joakime