Skip to content
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

ROB : cope with invalid length in streams #862

Closed
wants to merge 2 commits into from

Conversation

MartinThoma
Copy link
Member

Full credit to @pubpub-zz - this is just a rebased version of their branch: #861

MartinThoma and others added 2 commits May 7, 2022 08:39
As support for Python 3.5 and lower was dropped, we can use more modern syntax
issue #301

in case of invalid extract stream data looking for endstream
@MartinThoma MartinThoma changed the base branch from main to 2.0.0-dev May 7, 2022 06:42
@codecov
Copy link

codecov bot commented May 7, 2022

Codecov Report

Merging #862 (402a37e) into 2.0.0-dev (b580a45) will increase coverage by 0.13%.
The diff coverage is 90.90%.

@@              Coverage Diff              @@
##           2.0.0-dev     #862      +/-   ##
=============================================
+ Coverage      82.25%   82.38%   +0.13%     
=============================================
  Files             15       15              
  Lines           3640     3662      +22     
  Branches         781      787       +6     
=============================================
+ Hits            2994     3017      +23     
+ Misses           477      476       -1     
  Partials         169      169              
Impacted Files Coverage Δ
PyPDF2/merger.py 67.36% <ø> (ø)
PyPDF2/generic.py 86.15% <90.90%> (+0.12%) ⬆️
PyPDF2/_reader.py 81.79% <0.00%> (+0.39%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b580a45...402a37e. Read the comment docs.

@MartinThoma
Copy link
Member Author

The rebase was actually not necessary due to the squashing commit :-)

@MartinThoma MartinThoma closed this May 7, 2022
@MartinThoma MartinThoma deleted the pubpub-zz-iss301a branch May 7, 2022 06:45
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