Skip to content

Conversation

mrifni
Copy link
Contributor

@mrifni mrifni commented Mar 7, 2019

This PR is the fix for issue #15972.
I have added few unit tests to make sure the changes works.

Describing the problem:
Servlet context path specified in the application.properties file doesnt remove leading or trailing spaces and and it may result in 404 errors.

@wilkinsona wilkinsona changed the title Bugfix trim context path Trim trailing whitespace from value of spring.server.servlet.context-path Mar 7, 2019
@wilkinsona wilkinsona added this to the 2.2.x milestone Mar 7, 2019
@wilkinsona wilkinsona added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Mar 7, 2019
@snicoll snicoll self-assigned this Mar 12, 2019
@snicoll snicoll closed this in b229010 Mar 12, 2019
snicoll added a commit that referenced this pull request Mar 12, 2019
* pr/16165:
  Polish "Trim trailing whitespace from spring.server.servlet.context-path"
  Trim trailing whitespace from spring.server.servlet.context-path
@snicoll snicoll modified the milestones: 2.2.x, 2.2.0.M2 Mar 12, 2019
@snicoll
Copy link
Member

snicoll commented Mar 12, 2019

@mrifni thank you very much for making your first contribution to Spring Boot. This is now merged in master with a polish commit.

@mrifni
Copy link
Contributor Author

mrifni commented Mar 13, 2019

@snicoll you welcome! happy to contribute.

@mrifni mrifni deleted the BUGFIX-TRIM_CONTEXT_PATH branch March 19, 2019 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants