Probably a good idea. I wrote this constructor when I thought Ninject would be doing the calling, but we call the constructor manually in most places so a string would be a better choice.