Skip to content

Conversation

@RomanDavydchuk
Copy link
Contributor

Summary

Add N8N_GIT_NODE_DISABLE_BARE_REPOS environment variable which disables support for bare repositories in Git Node when set to true. Also disable bare repositories altogether for cloud deployment type. Add warning on startup if the user hasn't specified any value for the environment variable

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/NODE-3614/remote-code-execution-via-git-node

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team node/improvement New feature or request labels Sep 15, 2025
@codecov
Copy link

codecov bot commented Sep 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@RomanDavydchuk RomanDavydchuk marked this pull request as ready for review September 15, 2025 17:57
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 6 files

Prompt for AI agents (all 1 issues)

Understand the root cause of the following 1 issues and fix them.


<file name="packages/nodes-base/nodes/Git/__test__/Git.node.test.ts">

<violation number="1" location="packages/nodes-base/nodes/Git/__test__/Git.node.test.ts:112">
Assert absence of &#39;safe.bareRepository=explicit&#39; instead of exact empty array to avoid brittle tests.</violation>
</file>

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

Copy link
Contributor

@michael-radency michael-radency left a comment

Choose a reason for hiding this comment

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

LGTM

@currents-bot
Copy link

currents-bot bot commented Sep 18, 2025

E2E Tests: n8n tests passed after 6m 17s

🟢 556 · 🔴 0 · ⚪️ 2

View Run Details

Run Details

  • Project: n8n

  • Groups: 3

  • Framework: Playwright

  • Run Status: Passed

  • Commit: 9428116

  • Spec files: 106

  • Overall tests: 558

  • Duration: 6m 17s

  • Parallelization: 6

Groups

GroupId Results Spec Files Progress
ui 🟢 297 · 🔴 0 · ⚪️ 2 61 / 61
ui:isolated 🟢 5 · 🔴 0 · ⚪️ 0 1 / 1
No name 🟢 254 · 🔴 0 · ⚪️ 0 44 / 44


This message was posted automatically by currents.dev | Integration Settings

@RomanDavydchuk RomanDavydchuk merged commit 5bf3db5 into master Sep 18, 2025
24 checks passed
@RomanDavydchuk RomanDavydchuk deleted the node-3614-remote-code-execution-via-git-node branch September 18, 2025 15:33
@Joffcom
Copy link
Member

Joffcom commented Sep 22, 2025

Got released with [email protected]

pllusin pushed a commit to pllusin/n8n that referenced this pull request Sep 30, 2025
jason-schell pushed a commit that referenced this pull request Oct 2, 2025
…ow users to disable bare repositories in Git Node (#19559)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team node/improvement New feature or request Released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants