Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • slapos slapos
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 129
    • Merge requests 129
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • slaposslapos
  • Merge requests
  • !1645

Fix patches from Github

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Carlos Ramos Carreño requested to merge vnmabus/slapos:fix/github_patch_md5 into master Sep 24, 2024
  • Overview 4
  • Commits 1
  • Pipelines 0
  • Changes 3

Several components use patches directly from Github. Unfortunately, Github changes the number of characters displayed for the commits in index lines depending on the size of the repository. That means, that the content of these patches may change, invalidating the MD5.

By using the ?full_index=1 param (https://github.com/orgs/community/discussions/46034#discussioncomment-4846112) we can request that these patches include the full names of the commits, thus preventing future changes.

This has been previously discussed in https://lab.nexedi.com/nexedi/slapos/-/merge_requests/1630#note_216635

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/github_patch_md5
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7