There's no minProcessors anymore, only minSpareThreads. Similarly, maxProcessors should return maxThreads. You should add a note in the description that both are deprecated.
I don't use minProcessors or maxProcessors. I just left them in the descriptor instead of removing them. I can either add a note or remove them.
Forget it then :) Adding a "deprecated" note wouldn't be completely useless, I guess.
BTW, I think StandardServer should be adapted to create the new conector format, rather than use the old attribute names (and likely the socket factory also).
Can you elaborate?
Well, when you press commit, it saves back the server.xml file, right ?
I believe the Connector element it saves doesn't use the "modern" syntax (see the current server.xml). That should be esp true for SSL and JK.
Also, I have one big gripe with the save server.xml feature: it creates an unreadable server.xml, since it mindlessly saves every field on every element. Kinda crappy, and I'm pretty sure many people don't use the admin webapp because it trashes their server.xml (ok, maybe it's becuase of the bugs also ;-) ) ...
I think I have a way to fix this:
- when saving an element, figure out what's the associated className
- instantiate an instance of that class (it there's no default constructor, skip this algorithm)
- for every field, also retrieve the value from the "blank" object
- if the two values match, it means the field has its default value, so don't save it to server.xml; otherwise, save it as usual
Remy
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
