Skip to content

Fix error message in multi-host configuratiion#67

Open
r4nd0m wants to merge 1 commit into
lucee:masterfrom
r4nd0m:patch-1
Open

Fix error message in multi-host configuratiion#67
r4nd0m wants to merge 1 commit into
lucee:masterfrom
r4nd0m:patch-1

Conversation

@r4nd0m

@r4nd0m r4nd0m commented Oct 22, 2020

Copy link
Copy Markdown

When the tomcat is configured for multiple hosts, the following message is being displayed on startup:
"static path [/opt/lucee/web/] for servlet init param [lucee-web-directory] is not allowed. Path must use a web context specific placeholder"
Tha cause is a hardcoded lucee-web-directory.

When the tomcat is configured for multiple hosts, the following message is being displayed on startup:
"static path [/opt/lucee/web/] for servlet init param [lucee-web-directory] is not allowed. Path must use a web context specific placeholder"
Tha cause is a hardcoded lucee-web-directory.
@CLAassistant

CLAassistant commented Oct 22, 2020

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@cinco cinco left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thats ok, for me

@justincarter

justincarter commented Nov 27, 2020 via email

Copy link
Copy Markdown
Member

@r4nd0m

r4nd0m commented Nov 30, 2020

Copy link
Copy Markdown
Author

I think this would affect all existing web context paths (e.g. /opt/lucee/web/logs would become /opt/lucee/web/{web-context-label}/logs) so some users volume mappings could fail. We might need to solve this a slightly different way so that use cases for existing paths aren't impacted.

Good point @justincarter, how about adding:
<labels><label id="some_id_hash" name="./"/></labels>
to the /opt/lucee/server/lucee-server/context/lucee-server.xml
so we have a default label whose path will be resolved to the /opt/lucee/web/ in case we have only one context?

I tested it in my single-context container by renaming the label (without changing the path to the config file, it has been changed automatically) and it looks like that worked
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants