Skip to content

Fixed CBL Android 197 gzipped attachment#621

Merged
pasin merged 4 commits intomasterfrom
feature/issue_197_gzipped_attachment
May 13, 2015
Merged

Fixed CBL Android 197 gzipped attachment#621
pasin merged 4 commits intomasterfrom
feature/issue_197_gzipped_attachment

Conversation

@hideki
Copy link
Copy Markdown

@hideki hideki commented May 13, 2015

  • Fixed Attachment.getContent(). It causes recursive calls that does unzip twice.
  • Fixed attachments table access. Previous codes does not handle properly encoding and encoding_length columns.
  • Fixed storing Basic64 encoded gzipped attachment. It was unzipped before storing attachment.
  • Fixed push replication to set Content-Encoding for gzipped attachment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I knew now there is only None and GZip. It is a little safer to check against Gzip.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. I will fix it.

pasin added a commit that referenced this pull request May 13, 2015
…chment

Fixed CBL Android 197 gzipped attachment
@pasin pasin merged commit 69aa7c0 into master May 13, 2015
@pasin pasin deleted the feature/issue_197_gzipped_attachment branch May 13, 2015 19:52
@pasin pasin removed the review label May 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants