Skip to content
This repository has been archived by the owner on May 4, 2023. It is now read-only.

Specify configuration file in servlet init-parameters #153

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jalpedersen
Copy link

This patch allows one to specify the configuration file to use when C-L is running in a servlet-container.

@rnewson
Copy link
Owner

rnewson commented Feb 26, 2012

I definitely like the intent of the patch. There's at least one place that needs a fix, an InputStream is opened but not closed. Using init(ServletConfig) is obviously preferable to my constructor approach. I can get away with that only because I embed Jetty. Making C-L a pure WAR file with a jetty launch script would be a nice enhancement.

@jalpedersen
Copy link
Author

Oops, committed a fix so the inputstream is closed. Making it a pure WAR file with a jetty launch script shouldn't be too much of a hassle as it's more or less the way I'm running it now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants