https://bz.apache.org/bugzilla/show_bug.cgi?id=48674

--- Comment #10 from Mark Thomas <ma...@apache.org> ---
(In reply to Christopher Schultz from comment #9)
> This can be a dangerous feature, for a couple of reasons.
> 
> 1. A bad configuration or vulnerability in the host-manager allows a remote
> party to write to the filesystem, rather than just trash an in-memory
> configuration

The remote user can almost certainly deploy applications so it is pretty much
game over anyway.

> 2. Important information in the file may be overwritten inadvertently

I think Store config saves the old version with a timestamp.

> 3. NOP configuration information in the file (e.g. comments) will likely be
> lost when the file is saved

Price you pay...

> I had a look at the StoreConfig-based patch, and I must admit that I got
> lost in the whole architecture at the point that I started reading code in
> the o.a.c.storeconfig package. There is very little javadoc explaining what
> the heck is going on. It looks quite over-engineered and has a lot of code
> that looks very similar across classes.

Saving configuration is extremely tricky. There might be some clean-up possible
but my recollection from the last time if looked at the code was that it was
fundamentally sound.

Overall, I think this is the way to go.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to