New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Android 4.3 unable to upload HTTP attachments #1992

Closed
nolanlawson opened this Issue Apr 20, 2014 · 7 comments

Comments

Projects
None yet
3 participants
@nolanlawson
Copy link
Member

nolanlawson commented Apr 20, 2014

As the tests in #1937 show, older versions of Android (not that old, though!) have problems with HTTP attachments in the stock browser. Comparing desktop Firefox in the test "test some attachments" to my Galaxy Nexus, I see FX puts:

{ _id: 'bin_doc2',
  _rev: '2-393697643ece45d64435e5d2b8070357',
  _attachments: 
   { 'foo2.txt': 
      { content_type: 'text/plain',
        revpos: 2,
        digest: 'md5-oWU+DkMWYcPGKs/F0Lic5A==',
        length: 30,
        stub: true },
     'foo.txt': 
      { content_type: 'text/plain',
        revpos: 1,
        digest: 'md5-cCkGbCesb17xjWYNV0GXmg==',
        length: 0,
        stub: true } } }

whereas Android puts:

{ _id: 'bin_doc2',
  _rev: '2-36000100a8f491bad3d956f4911ba3e7',
  _attachments: 
   { 'foo2.txt': 
      { content_type: 'text/plain',
        revpos: 2,
        digest: 'md5-cCkGbCesb17xjWYNV0GXmg==',
        length: 0,
        stub: true },
     'foo.txt': 
      { content_type: 'text/plain',
        revpos: 1,
        digest: 'md5-cCkGbCesb17xjWYNV0GXmg==',
        length: 0,
        stub: true } } }

That length: 0 stands out like a sore thumb. It must have a problem uploading binary.

@nolanlawson

This comment has been minimized.

Copy link
Member

nolanlawson commented Apr 20, 2014

screenshot_2014-04-19-21-29-11-smaller

@nolanlawson

This comment has been minimized.

Copy link
Member

nolanlawson commented May 5, 2014

Confirmed that Android 4.4 does a PUT with the correct text/plain data, whereas 4.3 does not. Pcaps: https://www.dropbox.com/sh/a614b7m4ik5s56l/Mnwhad0hYx

nolanlawson added a commit that referenced this issue May 5, 2014

nolanlawson added a commit that referenced this issue May 5, 2014

nolanlawson added a commit that referenced this issue May 7, 2014

nolanlawson added a commit that referenced this issue May 7, 2014

@daleharvey

This comment has been minimized.

Copy link
Member

daleharvey commented May 7, 2014

@daleharvey daleharvey closed this May 7, 2014

@tilgovi

This comment has been minimized.

Copy link
Contributor

tilgovi commented May 7, 2014

@nolanlawson what tool is that screenshot from?

@daleharvey

This comment has been minimized.

Copy link
Member

daleharvey commented May 8, 2014

looks like http://html5test.com/ to me

@tilgovi

This comment has been minimized.

Copy link
Contributor

tilgovi commented May 8, 2014

Thanks!

@nolanlawson

This comment has been minimized.

Copy link
Member

nolanlawson commented May 8, 2014

Yup, it is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment