Wai Parse FilenameTooLong error incorrectly thrown for too long param name

[moll]

Hey,

I think the FilenameTooLong error in Wai Parse is incorrectly thrown for a too long key name, not the file name.

wai/wai-extra/Network/Wai/Parse.hs

Lines 583 to 589 in 53da27d

	Just (mct, name, Just filename) -> do
	case prboKeyLength o of
	Just maxKeyLength ->
	when (S.length name > maxKeyLength) $
	E.throwIO $
	FilenameTooLong name maxKeyLength
	Nothing -> return ()

The name length check mirrors the one just beneath the other Just (_ct, name, Nothing) case and that's for the parameter name. There's no code right now that checks the file name length, so the error itself seems unused.

[moll]

Now that I'm looking at it, the UrlEncoded variant of conduitRequestBodyEx doesn't check the parameter name lengths at all. Either both should or neither should. The current method of just checking param name lengths of multipart forms is odd. Doesn't gain nor protect much. I think just checking the header line length is sufficient to protect against metadata (such as param name, file name) resource abuse. Individual param name and filename length checks should be left to business logic.