Skip to content

Reason phrase for errors is bad #2378

@ietf-svn-bot

Description

@ietf-svn-bot

resolution_fixed type_defect | by mt@lowentropy.net


I've been testing out the https://datatracker.ietf.org/api/submit (which doesn't have a component BTW) and needless to say, I've generated a few bad uploads. The HTTP response body includes a nice message explaining the error, which is great.

However, the same message is added to the reason phrase.

The reason phrase can be set to a fixed string. No one actually uses it and it's been removed in HTTP/2 anyway; a long string like this probably only causes problems.

For example:
HTTP/1.1 400 Validation Error: [u"idnits 2.15.00 \n\n/a/www/www6s/staging/draft-thomson-upload-test2-draft-thomson-upload-test2-00.txt:\n\n  Showing Errors (**), Flaws (~~), Warnings (==), and Comments (--).\n  Errors MUST be fixe
Server: blah
Content-Length: ...

Validation Error: [u"idnits 2.15.00 \n\n/a/www/www6s/staging/draft-thomson-upload-test2-draft-thomson-upload-test2-00.txt:\n\n  Showing Errors (**), Flaws (~~), Warnings (==), and Comments (--).\n  Errors MUST be fixe...

Issue migrated from trac:2378 at 2022-03-04 05:58:10 +0000

Metadata

Metadata

Assignees

No one assigned

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions