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

ServletEndpoints should not register under the Jersey context path if there is a management port #15891

Closed
zuckel opened this issue Feb 11, 2019 · 0 comments
Assignees
Labels
type: bug A general bug
Milestone

Comments

@zuckel
Copy link

zuckel commented Feb 11, 2019

In Spring Boot 2.1.2 (since 2.0.7) with a Jersey webapp, a @ServletEndpoint will be registered under the jersey context-path even on the management port.

Current behavior

If the configuration looks like this,

@ApplicationPath("/app")
public class JerseyConfig extends ResourceConfig {
@ServletEndpoint(id = "selftest")
public class SelftestEndpoint implements Supplier<EndpointServlet> {
server.port: 8080
management.server.port: 8081

the actuator will be registered under http://localhost:8081/app/actuator/selftest.

The actuator overview http://localhost:8081/actuator lists the actuator under http://localhost:8081/actuator/selftest.

Expectations

  • All actuators should be reachable following the links from the actuator overview.
  • If there is a management port, all actuators should sit directly under the same actuator context-path.

Sample project

https://github.com/1and1/httpselftest/tree/servletendpoint (branch servletendpoint, sub-module selftest-springboot-example)

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 11, 2019
@mbhave mbhave added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Feb 11, 2019
@mbhave mbhave added this to the 2.1.x milestone Feb 11, 2019
@mbhave mbhave self-assigned this Feb 11, 2019
@mbhave mbhave modified the milestones: 2.1.x, 2.1.3 Feb 13, 2019
@mbhave mbhave closed this as completed in bbe555a Feb 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

No branches or pull requests

3 participants