Se upp med genererad toString()

För ett par dagar sen råkade jag och en kollega ut för en lite skrämmande sak. Vi satt och arbetade med loggningen i en webbapplikation.

I och med det så lät vi utvecklingsmiljön (IDEA IntelliJ) generera ett antal toString()-metoder åt oss för att öka tydligheten i loggmeddelandena. IntelliJ gör oftast riktigt snygga såna metoder. Men plötsligt upptäcker vi att lösenord skrivs ner i loggen för nyregistrerade användare! Förvisso bara i testmiljön men nog för att man ska få hjärtat i halsgropen.

Boven i dramat var en av våra genererade toString(). Ptja, hur skulle IntelliJ kunna veta att just password-attributet inte skulle med i utdata?

public class UserPutData {

private final String userName;

private final String password;

private final String email;

private final String firstname;

private final String surname;

private final String postcode;

...

@Override

public String toString() {

return "UserPutData{" +

"userName='" + userName + '\'' +

", password='" + password + '\'' +

", email='" + email + '\'' +

", firstName='" + firstname + '\'' +

", surname='" + surname + '\'' +

", postcode='" + postcode + '\'' +

'}';

}

}

Ooups!