Skip to content

http.server's send_error could set the content-length of the error body #60292

Closed

Description

@pitrou
BPO 16088
Nosy @jcea, @orsenthil, @pitrou
Files
  • http_error_content_length.patch
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = None
    closed_at = <Date 2012-10-11.06:19:07.606>
    created_at = <Date 2012-09-29.23:02:31.345>
    labels = ['type-feature', 'library']
    title = "http.server's send_error could set the content-length of the error body"
    updated_at = <Date 2012-10-11.06:19:07.604>
    user = 'https://github.com/pitrou'

    bugs.python.org fields:

    activity = <Date 2012-10-11.06:19:07.604>
    actor = 'orsenthil'
    assignee = 'none'
    closed = True
    closed_date = <Date 2012-10-11.06:19:07.606>
    closer = 'orsenthil'
    components = ['Library (Lib)']
    creation = <Date 2012-09-29.23:02:31.345>
    creator = 'pitrou'
    dependencies = []
    files = ['27349']
    hgrepos = []
    issue_num = 16088
    keywords = ['patch']
    message_count = 6.0
    messages = ['171597', '171598', '172520', '172612', '172622', '172623']
    nosy_count = 4.0
    nosy_names = ['jcea', 'orsenthil', 'pitrou', 'python-dev']
    pr_nums = []
    priority = 'low'
    resolution = 'fixed'
    stage = 'resolved'
    status = 'closed'
    superseder = None
    type = 'enhancement'
    url = 'https://bugs.python.org/issue16088'
    versions = ['Python 3.4']

    Activity

    1. pitrou commented on Sep 29, 2012

      @pitrou
      MemberAuthor

      BaseHTTPRequestHandler.send_error() sets "connection: close", so the client shouldn't need a content-length, but it can still be nicer to send it (e.g. it seems to solve an issue with wget, see bpo-15991).

    2. added
      stdlibStandard Library Python modules in the Lib/ directory
      type-featureA feature request or enhancement
      on Sep 29, 2012
    3. pitrou commented on Sep 29, 2012

      @pitrou
      MemberAuthor

      Here is a patch.

    4. orsenthil commented on Oct 9, 2012

      @orsenthil
      Member

      +1 to this change. It may be helpful in the cases like bpo-15991.

    5. jcea commented on Oct 11, 2012

      @jcea
      Member

      Looks good.

    6. python-dev commented on Oct 11, 2012

      python-devmannequin
      Mannequin

      New changeset ee34cb049a10 by Senthil Kumaran in branch 'default':
      Issue bpo-16088: BaseHTTPRequestHandler's send_error method includes a
      http://hg.python.org/cpython/rev/ee34cb049a10

    7. orsenthil commented on Oct 11, 2012

      @orsenthil
      Member

      I took the privilege of testing, adding news and docs and committing it.

      Thank you!

    8. transferred this issue fromon Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Metadata

    Metadata

    Assignees

    No one assigned

      Labels

      stdlibStandard Library Python modules in the Lib/ directorytype-featureA feature request or enhancement

      Projects

      No projects

        Milestone

        No milestone

        Relationships

        None yet

        Development

        No branches or pull requests

        Issue actions