We should by all means do String.trim on input values- from properties or otherwise.
On Tue, Feb 23, 2010 at 9:36 AM, Sig Rinde <[email protected]> wrote: > Ethan / Dick > > Noticed that the default.props file that was included in today's RC > indeed had much trailing characters (spaces) so even if I used vi to > add the role.api_test line to it I had to clean it up first. > > Unless you're aware of the issue that I stumbled into last you'll walk > into the trap. Space characters are not easy to see unless you poke > around looking for them... > > Suggest you clean up that file so it's free of that easy-to-fall-into-trap. > > Sig > > On 22 February 2010 14:37, Ethan Jewett <[email protected]> wrote: >> Whew, that's a relief :-) >> >> On Mon, Feb 22, 2010 at 1:10 AM, Sig Rinde <[email protected]> wrote: >>> Ethan, >>> >>> you were right! So much easier to find errors when one knows where to look >>> :) >>> >>> A single tailing space character after the rol.api_test line, sneaky >>> little invisible character... >>> >>> My fault, used copy past using textmate. Next time vi only... >
