Hi,

I have tested this on the App Engine app that was showing errors, and I no longer see the errors. Which is a good thing.

I'm not sure that my fix of catching any Exception is correct, because it is overly broad. We could catch the specific django.core.exceptions.DisallowedHost exception that is raised by request.build_absolute_uri(), but that would involve importing the exception from Django. Or we could catch Exception and check the doc string. But that then ties us to the docstring. Something like this:

try:
    request_url = request.build_absolute_uri()
except Exception as err:
    if err.__doc__ == "HTTP_HOST header contains invalid value":
        request_url = None
    else:
        raise

Also I figured setting request_url = None and letting the logging request complete with incomplete data was more useful than causing an uncaught exception further up the stack.