Just for consistency, shouldn't all the ```new``` calls be placed in this constructor ? I mean for the ```credentials```, ```attributes```...