Skip to content
New issue

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

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

Already on GitHub? # to your account

Disable caching for range requests #8716

Merged
merged 1 commit into from
Aug 5, 2020

Conversation

McSinyx
Copy link
Contributor

@McSinyx McSinyx commented Aug 5, 2020

This fixes corrupted shallow wheels when caching is involved (resolves GH-8701) until we know better on why range responses are not cached correctly. It would be really nice if this can land as part of the next bugfix release (20.2.2).

@sbidoul sbidoul added this to the 20.2.2 milestone Aug 5, 2020
@chrahunt chrahunt merged commit f51bd8f into pypa:master Aug 5, 2020
@pradyunsg
Copy link
Member

@McSinyx Could you file a PR adding a NEWS fragment for this PR and the corresponding issue -- basically "8701.bugfix" and "8716.bugfix" files containing the exact same text as support for this PR.

I think every bugfix that we're including in a bugfix release should be mentioned in the changelog. :)

@McSinyx
Copy link
Contributor Author

McSinyx commented Aug 7, 2020

I'm sorry for late response (I saw your comment just a few minutes after it was posted but decided to continue messing with GitHub Action instead 😅). TIL about the duplicate news entry handling, which is nice but I wonder why we need it for this case.

@McSinyx McSinyx deleted the fix-range-request-cache branch August 7, 2020 17:21
pradyunsg pushed a commit to pradyunsg/pip that referenced this pull request Aug 11, 2020
bors bot referenced this pull request in duckinator/emanate Aug 11, 2020
162: Update pip to 20.2.2 r=duckinator a=pyup-bot


This PR updates [pip](https://pypi.org/project/pip) from **20.2.1** to **20.2.2**.



<details>
  <summary>Changelog</summary>
  
  
   ### 20.2.2
   ```
   ===================

Bug Fixes
---------

- Only attempt to use the keyring once and if it fails, don&#39;t try again.
  This prevents spamming users with several keyring unlock prompts when they
  cannot unlock or don&#39;t want to do so. (`8090 &lt;https://github.com/pypa/pip/issues/8090&gt;`_)
- Fix regression that distributions in system site-packages are not correctly
  found when a virtual environment is configured with ``system-site-packages``
  on. (`8695 &lt;https://github.com/pypa/pip/issues/8695&gt;`_)
- Disable caching for range requests, which causes corrupted wheels
  when pip tries to obtain metadata using the feature ``fast-deps``. (`8701 &lt;https://github.com/pypa/pip/issues/8701&gt;`_, `8716 &lt;https://github.com/pypa/pip/issues/8716&gt;`_)
- Always use UTF-8 to read ``pyvenv.cfg`` to match the built-in ``venv``. (`8717 &lt;https://github.com/pypa/pip/issues/8717&gt;`_)
- 2020 Resolver: Correctly handle marker evaluation in constraints and exclude
  them if their markers do not match the current environment. (`8724 &lt;https://github.com/pypa/pip/issues/8724&gt;`_)
   ```
   
  
</details>


 

<details>
  <summary>Links</summary>
  
  - PyPI: https://pypi.org/project/pip
  - Changelog: https://pyup.io/changelogs/pip/
  - Homepage: https://pip.pypa.io/
</details>



Co-authored-by: pyup-bot <github-bot@pyup.io>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 10, 2021
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[2020-resolver & fast-deps] BadZipFile when force-reinstalling packages
4 participants