I'm not sure I like returning None. Is there a good reason not to rename this function `read_or_create_secret_key_file()` and have it do just that? If it can't read, or create and read, maybe it should raise an exception?