Uploaded image for project: 'Nuxeo Python Client'
  1. Nuxeo Python Client
  2. NXPY-59

Ensure the retreived data length respects the Content-Length HTTP header

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Detail
    • Resolution: Won't Fix
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.0
    • Component/s: None
    • Tags:
    • Upgrade notes:
      Hide

      After a first patch and team discussion, this issue is not adressable: too complexe to handle every cases, too many time for something that will be fixed with requests 3). So closing this, it will help us understand eventual corner case bug.
      Warning: issue to mitigate as there never has those kind of server issues. And as the client will run against Nuxeo only, we are not facing several server types that may eventually have such issues.

      Show
      After a first patch and team discussion, this issue is not adressable: too complexe to handle every cases, too many time for something that will be fixed with requests 3). So closing this, it will help us understand eventual corner case bug. Warning: issue to mitigate as there never has those kind of server issues. And as the client will run against Nuxeo only, we are not facing several server types that may eventually have such issues.
    • Sprint:
      nxDrive 10.2.3
    • Story Points:
      1

      Description

      An interesting reading: On Incomplete HTTP Reads and the Requests Library In Python.

      To summarize, the Requests module will not raise an exception when the retreived data length is not equal to what we should get as defined by the Content-Length HTTP header.

      There is some work on the urllib3 and Requests parts, but as there is no evidence when they will change that behavior (if they decide to do it). So we need to add the check on our side.

        Attachments

          Issue Links

          is related to

          Clean up - NXPY-60 Remove the Content Length check for requests 2.x

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

            Activity

              People

              • Assignee:
                mschoentgen Mickaël Schoentgen
                Reporter:
                mschoentgen Mickaël Schoentgen
                Participants:
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 6 hours, 30 minutes
                  6h 30m