admin/n8n-enterprise-unlocked
1
0
Fork 0
mirror of https://github.com/Abdulazizzn/n8n-enterprise-unlocked.git synced 2025-12-21 03:42:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

feat(LangChain Nodes): Upgrade LangChain and improve proxy handling for LLM nodes (#16778)

Browse Source
This commit is contained in:
Eugene
2025-07-03 14:22:25 +02:00
committed by GitHub
parent c1ac8c19ef
commit cfe3699782
18 changed files with 368 additions and 299 deletions
Download Patch File Download Diff File Expand all files Collapse all files

165
packages/@n8n/nodes-langchain/utils/tests/httpProxyAgent.test.ts
View File

@@ -1,13 +1,13 @@
import { HttpsProxyAgent } from 'https-proxy-agent';
import { ProxyAgent } from 'undici';
import { getHttpProxyAgent } from '../httpProxyAgent';
import { getProxyAgent } from '../httpProxyAgent';
// Mock the https-proxy-agent package
jest.mock('https-proxy-agent', () => ({
HttpsProxyAgent: jest.fn().mockImplementation((url) => ({ proxyUrl: url })),
// Mock the dependencies
jest.mock('undici', () => ({
ProxyAgent: jest.fn().mockImplementation((url) => ({ proxyUrl: url })),
}));
describe('getHttpProxyAgent', () => {
describe('getProxyAgent', () => {
// Store original environment variables
const originalEnv = { ...process.env };
@@ -19,6 +19,8 @@ describe('getHttpProxyAgent', () => {
delete process.env.http_proxy;
delete process.env.HTTPS_PROXY;
delete process.env.https_proxy;
delete process.env.NO_PROXY;
delete process.env.no_proxy;
});
// Restore original environment after all tests
@@ -26,63 +28,130 @@ describe('getHttpProxyAgent', () => {
process.env = originalEnv;
});
it('should return undefined when no proxy environment variables are set', () => {
const agent = getHttpProxyAgent();
expect(agent).toBeUndefined();
expect(HttpsProxyAgent).not.toHaveBeenCalled();
describe('target URL not provided', () => {
it('should return undefined when no proxy environment variables are set', () => {
const agent = getProxyAgent();
expect(agent).toBeUndefined();
expect(ProxyAgent).not.toHaveBeenCalled();
});
it('should create ProxyAgent when HTTPS_PROXY is set', () => {
const proxyUrl = 'https://proxy.example.com:8080';
process.env.HTTPS_PROXY = proxyUrl;
const agent = getProxyAgent();
expect(agent).toEqual({ proxyUrl });
expect(ProxyAgent).toHaveBeenCalledWith(proxyUrl);
});
it('should create ProxyAgent when https_proxy is set', () => {
const proxyUrl = 'https://proxy.example.com:8080';
process.env.https_proxy = proxyUrl;
const agent = getProxyAgent();
expect(ProxyAgent).toHaveBeenCalledWith(proxyUrl);
expect(agent).toEqual({ proxyUrl });
});
it('should respect priority order of proxy environment variables', () => {
// Set multiple proxy environment variables
process.env.HTTP_PROXY = 'http://http-proxy.example.com:8080';
process.env.http_proxy = 'http://http-proxy-lowercase.example.com:8080';
process.env.HTTPS_PROXY = 'https://https-proxy.example.com:8080';
process.env.https_proxy = 'https://https-proxy-lowercase.example.com:8080';
const agent = getProxyAgent();
// Should use https_proxy as it has highest priority now
expect(ProxyAgent).toHaveBeenCalledWith('https://https-proxy-lowercase.example.com:8080');
expect(agent).toEqual({ proxyUrl: 'https://https-proxy-lowercase.example.com:8080' });
});
});
it('should create HttpsProxyAgent when HTTP_PROXY is set', () => {
const proxyUrl = 'http://proxy.example.com:8080';
process.env.HTTP_PROXY = proxyUrl;
describe('target URL provided', () => {
it('should return undefined when no proxy is configured', () => {
const agent = getProxyAgent('https://api.openai.com/v1');
const agent = getHttpProxyAgent();
expect(agent).toBeUndefined();
expect(ProxyAgent).not.toHaveBeenCalled();
});
expect(HttpsProxyAgent).toHaveBeenCalledWith(proxyUrl);
expect(agent).toEqual({ proxyUrl });
});
it('should create ProxyAgent for HTTPS URL when HTTPS_PROXY is set', () => {
const proxyUrl = 'https://proxy.example.com:8080';
process.env.HTTPS_PROXY = proxyUrl;
it('should create HttpsProxyAgent when http_proxy is set', () => {
const proxyUrl = 'http://proxy.example.com:8080';
process.env.http_proxy = proxyUrl;
const agent = getProxyAgent('https://api.openai.com/v1');
const agent = getHttpProxyAgent();
expect(agent).toEqual({ proxyUrl });
expect(ProxyAgent).toHaveBeenCalledWith(proxyUrl);
});
expect(HttpsProxyAgent).toHaveBeenCalledWith(proxyUrl);
expect(agent).toEqual({ proxyUrl });
});
it('should create ProxyAgent for HTTP URL when HTTP_PROXY is set', () => {
const proxyUrl = 'http://proxy.example.com:8080';
process.env.HTTP_PROXY = proxyUrl;
it('should create HttpsProxyAgent when HTTPS_PROXY is set', () => {
const proxyUrl = 'http://proxy.example.com:8080';
process.env.HTTPS_PROXY = proxyUrl;
const agent = getProxyAgent('http://api.example.com');
const agent = getHttpProxyAgent();
expect(agent).toEqual({ proxyUrl });
expect(ProxyAgent).toHaveBeenCalledWith(proxyUrl);
});
expect(HttpsProxyAgent).toHaveBeenCalledWith(proxyUrl);
expect(agent).toEqual({ proxyUrl });
});
it('should use HTTPS_PROXY for HTTPS URLs even when HTTP_PROXY is set', () => {
const httpProxy = 'http://http-proxy.example.com:8080';
const httpsProxy = 'https://https-proxy.example.com:8443';
process.env.HTTP_PROXY = httpProxy;
process.env.HTTPS_PROXY = httpsProxy;
it('should create HttpsProxyAgent when https_proxy is set', () => {
const proxyUrl = 'http://proxy.example.com:8080';
process.env.https_proxy = proxyUrl;
const agent = getProxyAgent('https://api.openai.com/v1');
const agent = getHttpProxyAgent();
expect(agent).toEqual({ proxyUrl: httpsProxy });
expect(ProxyAgent).toHaveBeenCalledWith(httpsProxy);
});
expect(HttpsProxyAgent).toHaveBeenCalledWith(proxyUrl);
expect(agent).toEqual({ proxyUrl });
});
it('should respect NO_PROXY for localhost', () => {
const proxyUrl = 'http://proxy.example.com:8080';
process.env.HTTP_PROXY = proxyUrl;
process.env.NO_PROXY = 'localhost,127.0.0.1';
it('should respect priority order of proxy environment variables', () => {
// Set multiple proxy environment variables
process.env.HTTP_PROXY = 'http://http-proxy.example.com:8080';
process.env.http_proxy = 'http://http-proxy-lowercase.example.com:8080';
process.env.HTTPS_PROXY = 'http://https-proxy.example.com:8080';
process.env.https_proxy = 'http://https-proxy-lowercase.example.com:8080';
const agent = getProxyAgent('http://localhost:3000');
const agent = getHttpProxyAgent();
expect(agent).toBeUndefined();
expect(ProxyAgent).not.toHaveBeenCalled();
});
// Should use HTTPS_PROXY as it has highest priority now
expect(HttpsProxyAgent).toHaveBeenCalledWith('http://https-proxy.example.com:8080');
expect(agent).toEqual({ proxyUrl: 'http://https-proxy.example.com:8080' });
it('should respect NO_PROXY wildcard patterns', () => {
const proxyUrl = 'http://proxy.example.com:8080';
process.env.HTTPS_PROXY = proxyUrl;
process.env.NO_PROXY = '*.internal.company.com,localhost';
const agent = getProxyAgent('https://api.internal.company.com');
expect(agent).toBeUndefined();
expect(ProxyAgent).not.toHaveBeenCalled();
});
it('should use proxy for URLs not in NO_PROXY', () => {
const proxyUrl = 'http://proxy.example.com:8080';
process.env.HTTPS_PROXY = proxyUrl;
process.env.NO_PROXY = 'localhost,127.0.0.1';
const agent = getProxyAgent('https://api.openai.com/v1');
expect(agent).toEqual({ proxyUrl });
expect(ProxyAgent).toHaveBeenCalledWith(proxyUrl);
});
it('should handle mixed case environment variables', () => {
const proxyUrl = 'http://proxy.example.com:8080';
process.env.https_proxy = proxyUrl;
process.env.no_proxy = 'localhost';
const agent = getProxyAgent('https://api.openai.com/v1');
expect(agent).toEqual({ proxyUrl });
expect(ProxyAgent).toHaveBeenCalledWith(proxyUrl);
});
});
});