Why do we use String here instead of &str? I feel like using &str when we don't need to own or mutate string is more convenient in a function parameter, but maybe it isn't a good practice. @varun is there a good reason for using String here?

nit: Just so you know since Error impls From<DatabaseError>, you don't need the explicit map_err. ? will do that for you.