Why do you have condition there? ```nil``` and ```''``` are both evaluated to empty string in erb. Only case that you solve by this condition is a case when ```@host.parameters['preseed-proxy']``` is false. Was this the intention? Otherwise I think that's cool.