//do not really need username since you set this value, not sure why it is returned
//String userName = mf.addPerson(person, userSignOn);
//changed above line to this since username already a variable name
//userName = mf.addPerson(person, userSignOn);
userName = mf.addPerson(person);
logger.log(Level.FINER, "Person " + userName + " has been persisted");
// retrieve again ???
//person=mf.getPerson(userName);
// login person
SecurityHandler.getInstance().setLoggedInPerson(request, person);