Skip to content

Fix user git access#4253

Open
rasapala wants to merge 11 commits into
mainfrom
CVS-187604_user_download
Open

Fix user git access#4253
rasapala wants to merge 11 commits into
mainfrom
CVS-187604_user_download

Conversation

@rasapala
Copy link
Copy Markdown
Collaborator

@rasapala rasapala commented May 28, 2026

🛠 Summary

JIRA CVS-187604
Disables libgit2 owner validation for repositories opened by OVMS.
Clears selected libgit2 git config search paths to avoid reading host/user git configuration.
Disables pack .keep file checks to reduce stat calls during repository operations.

🧪 Checklist

  • Unit tests added.
  • The documentation updated.
  • Change follows security best practices.
    ``

Copilot AI review requested due to automatic review settings May 28, 2026 09:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates libgit2 initialization for Hugging Face git-clone based model pulls to avoid repository access failures when model directories are owned by a different OS user.

Changes:

  • Disables libgit2 owner validation for repositories opened by OVMS.
  • Clears selected libgit2 git config search paths to avoid reading host/user git configuration.
  • Disables pack .keep file checks to reduce stat calls during repository operations.

Comment thread src/pull_module/libgit2.cpp
Comment thread src/pull_module/libgit2.cpp
@rasapala rasapala requested review from dtrawins and mzegla May 28, 2026 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants