'request.logger' > 'request.app.logger' in StaticFileMixin

[samuelcolvin]

(I believe this error exists on master too, but i'm using 0.21 as base as per the instructions).

What these changes does?

prevent 500 exception with extremely long url requests to the static file router.

This is a slightly ugly way of getting the error to occur, presumably there are other causes of a suitable exception, but this how I found the error and seems to demonstrate the point.

The code causing the error was not previously covered.

How to test your changes?

see test below

Checklist

• Code is written and well
• Tests for the changes are provided
• Documentation reflects the changes

[coveralls]

Coverage decreased (-1.1%) to 97.262% when pulling 82e7d45 on samuelcolvin:static-file-error-fix into 63cf125 on KeepSafe:0.21.

[coveralls]

Coverage increased (+0.05%) to 98.454% when pulling c9b7169 on samuelcolvin:static-file-error-fix into 63cf125 on KeepSafe:0.21.

[samuelcolvin]

@asvetlov any chance of merging this? I see it's still broken in master too.

[asvetlov]

Thanks

[lock]

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.