Skip to content

Conversation

@yumosx
Copy link

@yumosx yumosx commented Jan 15, 2026

Description

Add support for disabling SSL verification when verifySsl flag is false by providing a custom HTTPS agent. This allows connections to servers with self-signed certificates or in development environments where SSL verification needs to be bypassed.

ref: #6215

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Continue Tasks

Status Task Actions
▶️ Queued Update docs on PR View

Powered by Continue


Summary by cubic

Adds an optional SSL bypass for MCP SSE and HTTP client transports. When requestOptions.verifySsl is false, connections to self-signed or dev servers work; default behavior remains unchanged.

  • New Features
    • Use https.Agent({ rejectUnauthorized: false }) when verifySsl is false for SSE and streamable-http transports.
    • No agent is created when verifySsl is true or unset.
    • Added tests covering both transports and verifySsl states.

Written for commit 173b9db. Summary will update on new commits.

Add support for disabling SSL verification when verifySsl flag is false by providing a custom HTTPS agent. This allows connections to servers with self-signed certificates or in development environments where SSL verification needs to be bypassed.
@yumosx yumosx requested a review from a team as a code owner January 15, 2026 07:34
@yumosx yumosx removed the request for review from a team January 15, 2026 07:34
@yumosx yumosx requested a review from sestinj January 15, 2026 07:34
@continue
Copy link
Contributor

continue bot commented Jan 15, 2026

All Green - Keep your PRs mergeable

Learn more

All Green is an AI agent that automatically:

✅ Addresses code review comments

✅ Fixes failing CI checks

✅ Resolves merge conflicts


Unsubscribe from All Green comments

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jan 15, 2026
@github-actions
Copy link

github-actions bot commented Jan 15, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@yumosx
Copy link
Author

yumosx commented Jan 15, 2026

I have read the CLA Document and I hereby sign the CLA

const sseAgent =
serverConfig.requestOptions?.verifySsl === false
? new HttpsAgent({ rejectUnauthorized: false })
: undefined;

Check failure

Code scanning / CodeQL

Disabling certificate validation High

Disabling certificate validation is strongly discouraged.
const streamableAgent =
serverConfig.requestOptions?.verifySsl === false
? new HttpsAgent({ rejectUnauthorized: false })
: undefined;

Check failure

Code scanning / CodeQL

Disabling certificate validation High

Disabling certificate validation is strongly discouraged.
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.

No issues found across 2 files

@yumosx
Copy link
Author

yumosx commented Jan 15, 2026

@RomneyDa @sestinj I noticed that the lint errors were not introduced by this PR. I am more than happy to fix these lints.

> @continuedev/cli@0.0.0-dev lint
> tsc --noEmit && eslint .


/home/runner/work/continue/continue/extensions/cli/src/services/MCPService.ts
Error:   622:1  error  File has too many lines (515). Maximum allowed is 500  max-lines

 1 problem (1 error, 0 warnings)

Error: Process completed with exit code 1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant