Title: Replace use of mktemp by EliahKagan · Pull Request #56 · gitpython-developers/smmap · GitHub
Open Graph Title: Replace use of mktemp by EliahKagan · Pull Request #56 · gitpython-developers/smmap
X Title: Replace use of mktemp by EliahKagan · Pull Request #56 · gitpython-developers/smmap
Description: Fixes #41 This uses NamedTemporaryFile with delete=False to replace the one use of the deprecated mktemp function in smmap (reported in #41). This avoids the race condition inherent to mktemp, as the file is named and created together in a way that is effectively atomic. Because NamedTemporaryFile is not being used to automatically delete the file, it use and cleanup are unaffected by the change. (This PR is conceptually related to gitpython-developers/GitPython#1770.)
Open Graph Description: Fixes #41 This uses NamedTemporaryFile with delete=False to replace the one use of the deprecated mktemp function in smmap (reported in #41). This avoids the race condition inherent to mktemp, as t...
X Description: Fixes #41 This uses NamedTemporaryFile with delete=False to replace the one use of the deprecated mktemp function in smmap (reported in #41). This avoids the race condition inherent to mktemp, as t...
Opengraph URL: https://github.com/gitpython-developers/smmap/pull/56
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:e435ba1d-7a5b-2823-e224-4c1137d6f389 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | C46A:1327C8:1AEFB5:2593A5:6968C208 |
| html-safe-nonce | f4cd7fb67d7f43900e7e0de132fc8c80b9b678767764775e348d0a833b80a77f |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDNDZBOjEzMjdDODoxQUVGQjU6MjU5M0E1OjY5NjhDMjA4IiwidmlzaXRvcl9pZCI6IjY3MjY2MDQ4Njk2NzUzMDM0MzIiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 11042255708060fa0120eda83f3068b98442b24b06c039dde3b8f6c8ac79a5e8 |
| hovercard-subject-tag | pull_request:1641964251 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/gitpython-developers/smmap/pull/56/files |
| twitter:image | https://avatars.githubusercontent.com/u/1771172?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/1771172?s=400&v=4 |
| og:image:alt | Fixes #41 This uses NamedTemporaryFile with delete=False to replace the one use of the deprecated mktemp function in smmap (reported in #41). This avoids the race condition inherent to mktemp, as t... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | fdc7c66bd36a6c12eb8e771e806db863266e573fc299e77f27505a768d4f8a98 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/gitpython-developers/smmap git https://github.com/gitpython-developers/smmap.git |
| octolytics-dimension-user_id | 503709 |
| octolytics-dimension-user_login | gitpython-developers |
| octolytics-dimension-repository_id | 1865144 |
| octolytics-dimension-repository_nwo | gitpython-developers/smmap |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 1865144 |
| octolytics-dimension-repository_network_root_nwo | gitpython-developers/smmap |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 3223a6503d318917691422cdadfbe16cd8fb21e5 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width