gh-84481: Add ZipFile.data_offset attribute #132165
Conversation
This attribute provides the offset to zip data from the start of the file, when available.
|
All commit authors signed the Contributor License Agreement. |
|
I have signed the CLA with my current email. (I had already signed it under another email anyway) |
| except (AttributeError, OSError): | ||
| self.fp = _Tellable(self.fp) | ||
| self.start_dir = 0 | ||
| self._seekable = False | ||
| self._data_offset = None |
There was a problem hiding this comment.
It may not be set if mode == 'a' and if BadZipFile is raised. In this case, we should still have a None according to the docs (otherwise an AttributeError will be raised).
For mode 'r', it should be noted that failing _RealGetContents() means that the object will never be initialized so we don't care about the lack of attribute.
There was a problem hiding this comment.
I believe that in this case, since mode 'a' calls tell() unconditionally, we can set the data_offset to the result of that, right?
There was a problem hiding this comment.
Though honestly the defensive thing to do here is to initialize it to None early on then change it later when possible.
There was a problem hiding this comment.
Though honestly the defensive thing to do here is to initialize it to None early on then change it later when possible.
Yes, that's what I thought.
* Add ZipFile.data_offset attribute This attribute provides the offset to zip data from the start of the file, when available. * Add blurb-it * Try fixing class ref in NEWS
This attribute provides the offset to zip data from the start of the file, when available.
I chose
data_offsetto avoid confusion with the_offsetproperty that is unrelated.📚 Documentation preview 📚: https://cpython-previews--132165.org.readthedocs.build/