[PR #281] [MERGED] Fix a leaking file descriptor. #1108

New issue

Closed

opened 2026-03-01 14:48:27 +03:00 by kerem · 0 comments

kerem commented

2026-03-01 14:48:27 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ArchiveBox/ArchiveBox/pull/281
Author: @Julian
Created: 10/13/2019
Status: ✅ Merged
Merged: 10/21/2019
Merged by: @pirate

Base: master ← Head: master

📝 Commits (2)

eb641b3 Fix a leaking file descriptor.
2e4b506 Update archivebox/index.py

📊 Changes

1 file changed (+2 additions, -1 deletions)

View changed files

📝 archivebox/index.py (+2 -1)

📄 Description

Summary

Handles a ResourceWarning on Py3, and generally not leaking FDs especially on non-CPython.

Changes these areas

Bugfixes
Feature behavior
Command line interface
Configuration options
Internal architecture
Archived data layout on disk

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ArchiveBox/ArchiveBox/pull/281 **Author:** [@Julian](https://github.com/Julian) **Created:** 10/13/2019 **Status:** ✅ Merged **Merged:** 10/21/2019 **Merged by:** [@pirate](https://github.com/pirate) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`eb641b3`](https://github.com/ArchiveBox/ArchiveBox/commit/eb641b3e358fdbf21ea3d93f630571253286408c) Fix a leaking file descriptor. - [`2e4b506`](https://github.com/ArchiveBox/ArchiveBox/commit/2e4b506c21adcfc29253b163520d30e0bce8a7ae) Update archivebox/index.py ### 📊 Changes **1 file changed** (+2 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `archivebox/index.py` (+2 -1) </details> ### 📄 Description # Summary Handles a `ResourceWarning` on Py3, and generally not leaking FDs especially on non-CPython. # Changes these areas - [x] Bugfixes - [ ] Feature behavior - [ ] Command line interface - [ ] Configuration options - [ ] Internal architecture - [ ] Archived data layout on disk --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>