Skip to content

Conversation

ralenmdp
Copy link

@ralenmdp ralenmdp commented Apr 13, 2018

#12805

Try to look for the webapps folder so that WhitelabelErrorViewConfiguration will not create another error bean view and let InternalResourceViewResolver resolve the error page in embedded container.

@pivotal-issuemaster
Copy link

@ralenmdp Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@ralenmdp Thank you for signing the Contributor License Agreement!

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Apr 13, 2018
@philwebb
Copy link
Member

Thanks for the PR but I don't think this is the right way to fix the problem. I've not keen for JspTemplateAvailabilityProvider to have any special knowledge of the src/main/webapp folder structure that's used by the build system.

@philwebb philwebb closed this Apr 13, 2018
@philwebb philwebb added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged labels Apr 13, 2018
@wilkinsona
Copy link
Member

On reflection, we can't think of a better way to do this. The embedded containers already have knowledge or src/main/webapp and there's no easy way to pass that knowledge from the container to the availability provider. Hard-coding it seems like the best option.

@wilkinsona wilkinsona reopened this Jun 21, 2018
@wilkinsona wilkinsona added type: bug A general bug and removed status: declined A suggestion or change that we don't feel we should currently apply labels Jun 21, 2018
@wilkinsona wilkinsona added this to the 1.5.x milestone Jun 21, 2018
@wilkinsona wilkinsona self-assigned this Jun 21, 2018
@wilkinsona wilkinsona modified the milestones: 1.5.x, 1.5.15 Jun 21, 2018
@wilkinsona wilkinsona changed the title Fixes gh-12805 Custom error.jsp page is not used when app is run as anything other than a packaged war Jun 21, 2018
wilkinsona added a commit that referenced this pull request Jun 21, 2018
* gh-12859:
  Polish "Fix JSP availability check when not running as a packaged war"
  Fix JSP availability check when not running as a packaged war
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

Successfully merging this pull request may close these issues.

5 participants