fix: protect read file from path traversal

[ndom91]

☕️ Reasoning

• Ensure that the new get_pr_template_contents fn can't read anything outside of the project's root directory

🧢 Changes

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
gitbutler-web	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 22, 2024 5:07pm

[ndom91]

@mtsgrd some small rust question:

• There's probably no need for .context() on each method that potentially fails, right?
• Is it okay to add the entire git2 crate to gitbutler-fs's Cargo.toml? Can I add something like feature = { Repository } to only add what I need here?

[Byron]

• Is it okay to add the entire git2 crate to gitbutler-fs's Cargo.toml? Can I add something like feature = { Repository } to only add what I need here?

Features are predefined in the crate and they usually only provide features if it grates off other dependencies, not just code. When I see the example above, it reminds me of typescript imports :).