admin@glassfish.java.net

Re: asadmin vs security manager during an upgrade

From: Bobby Bissett <bobby.bissett_at_oracle.com>
Date: Thu, 6 May 2010 13:05:16 -0400

On May 5, 2010, at 7:54 PM, Bill Shannon wrote:
> Why don't you just ignore the exception from System.getenv?
> We're not expecting these environment variables to be set in
> the normal case, are we?

I can't answer those questions any better than you, :)

> It looks like the code just tries
> to find it as a System property, and if that fails tried to
> find it as an environment variable as some sort of fallback.

Really, I was just getting confirmation that anyone cared before
looking at it further. If it's the case that the property really isn't
needed, then ignoring the exception is fine. Not making the call in
the first place would be better, but maybe there's a use case for it.
Not my place to say, but I'll give it a shot and see where we get.

FWIW, this isn't an issue with the server.policy file as far as I can
tell. I think this code is operating before the policy system has been
started.

Cheers,
Bobby