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.

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

Default provider [files] section always executed #6049

Closed
sbordet opened this issue Mar 9, 2021 · 0 comments · Fixed by #6051
Closed

Default provider [files] section always executed #6049

sbordet opened this issue Mar 9, 2021 · 0 comments · Fixed by #6051
Assignees

Comments

@sbordet
Copy link
Contributor

sbordet commented Mar 9, 2021

Jetty version
10.0.x

Description
java -jar start.jar --add-modules=http,logging-logback copies resources/jetty-logging.properties, but it should not.

Interestingly,
java -jar start.jar --add-modules=logging-logback,http does not copy resources/jetty-logging.properties.

Since an explicit provider has been configured, the default provider should be out of the picture.

@sbordet sbordet self-assigned this Mar 9, 2021
sbordet added a commit that referenced this issue Mar 9, 2021
Keeping only enabled modules when processing the modules,
so that default provider modules don't get processed.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
sbordet added a commit that referenced this issue Apr 8, 2021
Keeping only enabled modules when processing the modules,
so that default provider modules don't get processed.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
sbordet added a commit that referenced this issue Apr 8, 2021
Updates after review.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
sbordet added a commit that referenced this issue Apr 11, 2021
Updates after review.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
sbordet added a commit that referenced this issue Apr 12, 2021
* Fixes #6049 - Default provider [files] section always executed

Keeping only enabled modules when processing the modules,
so that default provider modules don't get processed.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant