github-starred/n8n
2
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-12-05 19:27:26 -06:00
Code Issues 1.3k Packages Projects Releases 100 Wiki Activity

[PR #21999] support http proxy for http github option in source control #21991

New Issue
Open
opened 2025-11-20 06:22:07 -06:00 by GiteaMirror · 0 comments
GiteaMirror commented 2025-11-20 06:22:07 -06:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/n8n-io/n8n/pull/21999
Author: @minimorph223
Created: 11/18/2025
Status: 🔄 Open

Base: master ← Head: master

📝 Commits (3)

  • 0b6d5c6 support http proxy for gitub connect
  • 8e228da Update source-control-git.service.ee.ts
  • e76f12b Merge branch 'master' into master

📊 Changes

1 file changed (+6 additions, -3 deletions)

View changed files

📝 packages/cli/src/environments.ee/source-control/source-control-git.service.ee.ts (+6 -3)

📄 Description

Summary

When we use source control and https github connect, it does not respect http_proxy environment.
https://github.com/n8n-io/n8n/issues/21418

Related Linear tickets, Github issues, and Community forum posts

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)

Note

Configures git https.proxy from environment for HTTPS source-control connections, ensuring proxy-aware fetch/pull/push.

  • Source Control (Git Service)
    • For https connections, derive proxy via proxy-from-env and set git config https.proxy for the repository URL.
    • Keeps credential helper setup; proxy is added alongside existing HTTPS config.

Written by Cursor Bugbot for commit e76f12b056. This will update automatically on new commits. Configure here.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/n8n-io/n8n/pull/21999 **Author:** [@minimorph223](https://github.com/minimorph223) **Created:** 11/18/2025 **Status:** 🔄 Open **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (3) - [`0b6d5c6`](https://github.com/n8n-io/n8n/commit/0b6d5c6d2c8f247cdd63291e9b6e6533a0087700) support http proxy for gitub connect - [`8e228da`](https://github.com/n8n-io/n8n/commit/8e228da08c951bb40db82dc7d59fea04a336a747) Update source-control-git.service.ee.ts - [`e76f12b`](https://github.com/n8n-io/n8n/commit/e76f12b056c9e92145e312bf72d871b35bf752db) Merge branch 'master' into master ### 📊 Changes **1 file changed** (+6 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `packages/cli/src/environments.ee/source-control/source-control-git.service.ee.ts` (+6 -3) </details> ### 📄 Description ## Summary When we use source control and https github connect, it does not respect http_proxy environment. https://github.com/n8n-io/n8n/issues/21418 ## Related Linear tickets, Github issues, and Community forum posts ## Review / Merge checklist - [ ] PR title and summary are descriptive. ([conventions](../blob/master/.github/pull_request_title_conventions.md)) <!-- **Remember, the title automatically goes into the changelog. Use `(no-changelog)` otherwise.** --> - [ ] [Docs updated](https://github.com/n8n-io/n8n-docs) or follow-up ticket created. - [ ] Tests included. <!-- A bug is not considered fixed, unless a test is added to prevent it from happening again. A feature is not complete without tests. --> - [ ] PR Labeled with `release/backport` (if the PR is an urgent fix that needs to be backported) <!-- CURSOR_SUMMARY --> --- > [!NOTE] > Configures git `https.proxy` from environment for HTTPS source-control connections, ensuring proxy-aware fetch/pull/push. > > - **Source Control (Git Service)** > - For `https` connections, derive proxy via `proxy-from-env` and set `git` config `https.proxy` for the repository URL. > - Keeps credential helper setup; proxy is added alongside existing HTTPS config. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit e76f12b056c9e92145e312bf72d871b35bf752db. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the
pull-request
 label 2025-11-20 06:22:07 -06:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Backlog
bug-report
Bump
Bumped
closed:duplicate
closed:enhancement/feature
closed:incomplete-template
closed:info
closed:non-english
closed:support-issue
closed:working-as-expected
Duplicate
in linear
In Review
n8n team
Needs Feedback
Needs Feedback:Unable to reproduce
node/improvement
node/issue
node/new
pull-request
Mirrored from GitHub Pull Request
Released
Stale
status:complete
status:in-linear
status:pending-assignment
status:team-assigned
team:adore
team:ai
team:cats
team:nodes
team:payday
triage:complete
triage:in-progress
triage:needs-info
triage:needs-reproduction
triage:pending
triage:stalled
ui
Upcoming Release
No Label
pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/n8n#21991
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?