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

QuickStartConfiguration cannot be mixed with contexts that do not have a WEB-INF/quickstart-web.xml #11260

Closed
joakime opened this issue Jan 11, 2024 · 1 comment · Fixed by #11261
Closed

QuickStartConfiguration cannot be mixed with contexts that do not have a WEB-INF/quickstart-web.xml #11260

joakime opened this issue Jan 11, 2024 · 1 comment · Fixed by #11261
Assignees
@joakime
Labels
Bug For general bugs on Jetty side Sponsored This issue affects a user with a commercial support agreement
Milestone
12.0.x

Comments

@joakime
Copy link
Contributor

joakime commented Jan 11, 2024

Jetty version(s)
10, 11, 12

Jetty Environment
Any

Java version/vendor (use: java -version)
Any

OS type/version
Any

Description
If QuickStartConfiguration is enabled, and you have a mix of contexts, some with WEB-INF/quickstart-web.xml and some without you get an IllegalStateException preventing deployment of those contexts that do not have or need a quickstart.
@joakime joakime added the Bug For general bugs on Jetty side label Jan 11, 2024
@joakime joakime added this to the 12.0.x milestone Jan 11, 2024
@joakime joakime self-assigned this Jan 11, 2024
joakime added a commit that referenced this issue Jan 11, 2024
@joakime
Issue #11260 - Allow QuickStartConfiguration to be used in mixed co…
d601560 
…ntexts environment where some do not have a WEB-INF/quickstart-web.xml

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
joakime added a commit that referenced this issue Jan 11, 2024
@joakime
Revert "Issue #11260 - Allow QuickStartConfiguration to be used in …
dc96d91 
…mixed contexts environment where some do not have a WEB-INF/quickstart-web.xml"

This reverts commit d601560.
@sbordet sbordet added the Sponsored This issue affects a user with a commercial support agreement label Jan 11, 2024
joakime added a commit that referenced this issue Jan 11, 2024
@joakime
Issue #11260 - Allow QuickStartConfiguration to be used in mixed co…
ecc5853 
…ntexts environment where some do not have a WEB-INF/quickstart-web.xml

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
@joakime joakime linked a pull request Jan 11, 2024 that will close this issue
Issue #11260 - Allow QuickStartConfiguration to be used in mixed contexts environment where some do not have a WEB-INF/quickstart-web.xml #11261
Merged
@joakime joakime moved this to 🏗 In progress in Jetty 12.0.6 FROZEN Jan 24, 2024
joakime added a commit that referenced this issue Jan 25, 2024
@joakime
Issue #11260 - Allow QuickStartConfiguration to be used in mixed co…
6fec10e 
…ntexts environment where some do not have a WEB-INF/quickstart-web.xml (#11261)

* QuickStartConfiguration updated

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
@joakime
Copy link
Contributor Author

joakime commented Jan 25, 2024

Merged into jetty-10.0.x up to jetty-12.0.x

@joakime joakime closed this as completed Jan 25, 2024
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Jetty 12.0.6 FROZEN Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joakime joakime

Labels
Bug For general bugs on Jetty side Sponsored This issue affects a user with a commercial support agreement
Projects
No open projects
Jetty 10.0.20 / 11.0.20 FROZEN (Post EOCD Release)
Status: ✅ Done
Jetty 12.0.6 FROZEN
Status: ✅ Done
Milestone
12.0.x
2 participants
@joakime @sbordet