Title: Initial Migration of Fuzz Tests & Integration Scripts From the OSS-Fuzz Project Repo by DaveLak · Pull Request #1901 · gitpython-developers/GitPython · GitHub
Open Graph Title: Initial Migration of Fuzz Tests & Integration Scripts From the OSS-Fuzz Project Repo by DaveLak · Pull Request #1901 · gitpython-developers/GitPython
X Title: Initial Migration of Fuzz Tests & Integration Scripts From the OSS-Fuzz Project Repo by DaveLak · Pull Request #1901 · gitpython-developers/GitPython
Description: As discussed in #1889, this PR introduces the changes necessary to migrate the OSS-Fuzz fuzz tests and configuration scripts from the OSS-Fuzz repository into GitPython's repo. Summary of Changes Most of the changes proposed here are the same as described in the discussion thread linked above. Beyond that, there are a few minor differences and potentially non-obvious details that I want to call out here: Directory Structure Initially, I proposed a single, flat "fuzzing/" directory at the top level, but when I moved the files, I felt that a few sub-directories would help with organization. The fuzzing/READEME.md file introduced in this PR has a section describing them and some additional documentation. Updates to Fuzz Targets The fuzz test files in this PR include the changes that were originally proposed in google/oss-fuzz#11763. Seed Data Part of efficiency improvements I proposed in google/oss-fuzz#11763 included the addition of dictionary files (described in the new README) and seed corpus zip files. I stored these in a repo that I maintain: https://github.com/DaveLak/oss-fuzz-inputs/tree/main/gitpython, because OSS-Fuzz has understandable concerns about seed data bloating the size of their repo. In this PR: I moved the dictionary files to this repo for improved visibility/easier updates But I left the corpra zip files in my https://github.com/DaveLak/oss-fuzz-inputs/tree/main/gitpython/corpra repo pending further discussion regarding where those should live in the long term. If this is undesirable for any reason, let me know and I'll remove them. Otherwise I'll raise the question in the discussion thread. Next Steps @Byron & @EliahKagan, please: Let me know what you think of the changes here. Take a look at the related PR I have open in the OSS-Fuzz repo: google/oss-fuzz#11803 and let me know if theres anything I should change. A comment from each of you acknowledging and approving the changes there is required before it can be merged. If either of you could assist me with validating that the fuzzing/ directory doesn't get picked up by the publishing process, I'd appreciate it! I did what I could to test it locally but I'm not familiar enough with the release process to be confident. Thanks! I'll address feedback any feedback as quickly as possible.
Open Graph Description: As discussed in #1889, this PR introduces the changes necessary to migrate the OSS-Fuzz fuzz tests and configuration scripts from the OSS-Fuzz repository into GitPython's repo. Summary of Chang...
X Description: As discussed in #1889, this PR introduces the changes necessary to migrate the OSS-Fuzz fuzz tests and configuration scripts from the OSS-Fuzz repository into GitPython's repo. Summary of C...
Opengraph URL: https://github.com/gitpython-developers/GitPython/pull/1901
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:3f697396-2b13-f8fb-3458-b33ad99ce4b7 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | D870:347111:15F52AC:1EFEE55:696907C3 |
| html-safe-nonce | 70a23cd2ec234294d1894272ebd91e45b60c92b2e8a0a6fd927874265d495b3a |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJEODcwOjM0NzExMToxNUY1MkFDOjFFRkVFNTU6Njk2OTA3QzMiLCJ2aXNpdG9yX2lkIjoiNTcyMzM4ODI0MjYyNjY3NjY3NSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 6db5324676d9c00fd3ce692861adb917df9d27bdd727054cfc9664a3e24facfe |
| hovercard-subject-tag | pull_request:1819221359 |
| 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/GitPython/pull/1901/files |
| twitter:image | https://avatars.githubusercontent.com/u/17415134?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/17415134?s=400&v=4 |
| og:image:alt | As discussed in #1889, this PR introduces the changes necessary to migrate the OSS-Fuzz fuzz tests and configuration scripts from the OSS-Fuzz repository into GitPython's repo. Summary of Chang... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | e6156bd4ef9f2dc8dadf4c49a8f7ed8532186388cef72eda3ccb9f0ab3b8cfca |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/gitpython-developers/GitPython git https://github.com/gitpython-developers/GitPython.git |
| octolytics-dimension-user_id | 503709 |
| octolytics-dimension-user_login | gitpython-developers |
| octolytics-dimension-repository_id | 1126087 |
| octolytics-dimension-repository_nwo | gitpython-developers/GitPython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 1126087 |
| octolytics-dimension-repository_network_root_nwo | gitpython-developers/GitPython |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | ee2210c3e58153aae53400c942f8a7b4bbb43ec4 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width