Request Logger: Don't Print "empty_string" to Logs

Now that the darklaunch flags are being removed from EC and HEC we're starting to see logs that show `darklaunch=empty_string` show up. I tracked it down to this piece of code that's translating `""` to `"empty_string"`. This PR fixes that and adds a few tests around how we format the error messages.

ping @seth @hosh