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

Don't stream folders. #342

Merged
merged 1 commit into from Aug 23, 2016

Conversation

Projects
None yet
1 participant
@dduponchel
Collaborator

dduponchel commented Aug 22, 2016

We used a different check between the stream flag and the presence of
the data descriptor record. A folder would get a wrong 'streamed' flag
but no data descriptor, confusing some archivers.

Fix #339.

Don't stream folders.
We used a different check between the stream flag and the presence of
the data descriptor record. A folder would get a wrong 'streamed' flag
but no data descriptor, confusing some archivers.

@dduponchel dduponchel merged commit 671e1e1 into Stuk:master Aug 23, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@dduponchel dduponchel deleted the dduponchel:issue_339 branch Aug 23, 2016

@dduponchel dduponchel referenced this pull request Aug 23, 2016

Merged

Release v3.1.2 #340

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