Skip to content

Conversation

@DevinWalker
Copy link

@DevinWalker DevinWalker commented Nov 11, 2025

Description

This PR adds a troubleshooting subsection to the Jurassic Tube documentation in the Docker README.

Changes

  • Added a new 'Troubleshooting Jurassic Tube' subsection documenting how to resolve cURL error 35 (SSL_ERROR_SYSCALL) when connecting Jetpack to WordPress.com
  • Provides two solutions:
    1. Updating Docker's proxy settings through the Docker Desktop UI
    2. Adding proxy configuration constants to wp-config.php
  • Includes instructions to restart Docker containers after making changes

Jetpack product discussion

  • Slack thread I started: p1762822694261659-slack-CDLH4C1UZ
  • Slack thread Nick Pagazani started: p1754045486753419-slack-C05Q5HSS013
  • P2 Post: pdWQjU-Ka-p2

Does this pull request change what data or activity we track or use?

No

Testing instructions:

  • Review newly added section within readme for typos or potential confusing language

Context

Automatticians sometimes encounter SSL connection errors when using Jurassic Tube to connect Jetpack to WordPress.com. This documentation helps resolve those issues quickly.

Add documentation for resolving cURL error 35 (SSL_ERROR_SYSCALL) when Automatticians connect Jetpack to WordPress.com via Jurassic Tube.

The new troubleshooting subsection provides two solutions:
1. Updating Docker's proxy settings through Docker Desktop UI
2. Adding proxy configuration to wp-config.php

Includes reminder to restart Docker containers after making changes.
@github-actions
Copy link
Contributor

github-actions bot commented Nov 11, 2025

Thank you for your PR!

When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:

  • ✅ Include a description of your PR changes.
  • ✅ Add a "[Status]" label (In Progress, Needs Review, ...).
  • ✅ Add a "[Type]" label (Bug, Enhancement, Janitorial, Task).
  • ✅ Add testing instructions.
  • ✅ Specify whether this PR includes any changes to data or privacy.
  • ✅ Add changelog entries to affected projects

This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖


Follow this PR Review Process:

  1. Ensure all required checks appearing at the bottom of this PR are passing.
  2. Make sure to test your changes on all platforms that it applies to. You're responsible for the quality of the code you ship.
  3. You can use GitHub's Reviewers functionality to request a review.
  4. When it's reviewed and merged, you will be pinged in Slack to deploy the changes to WordPress.com simple once the build is done.

If you have questions about anything, reach out in #jetpack-developers for guidance!

@github-actions github-actions bot added the [Status] Needs Author Reply We need more details from you. This label will be auto-added until the PR meets all requirements. label Nov 11, 2025
@DevinWalker DevinWalker added the [Status] Needs Review This PR is ready for review. label Nov 11, 2025
@github-actions github-actions bot removed the [Status] Needs Review This PR is ready for review. label Nov 11, 2025
@DevinWalker DevinWalker marked this pull request as ready for review November 11, 2025 22:18
@DevinWalker DevinWalker requested a review from Copilot November 11, 2025 22:18
Copilot finished reviewing on behalf of DevinWalker November 11, 2025 22:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a troubleshooting subsection to help Automatticians resolve SSL connection errors when using Jurassic Tube to connect Jetpack to WordPress.com in Docker environments.

  • Adds a new "Troubleshooting Jurassic Tube" subsection documenting cURL error 35 (SSL_ERROR_SYSCALL)
  • Provides two solutions: updating Docker's proxy settings via Docker Desktop UI, or adding proxy configuration constants to wp-config.php
  • Includes instructions for restarting Docker containers after configuration changes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

define( 'WP_PROXY_PORT', '8080' );
```

**Note:** Be sure to restart the Docker container after making the changes:
Copy link

Copilot AI Nov 11, 2025

Choose a reason for hiding this comment

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

The note mentions "the Docker container" (singular) but the commands jetpack docker down and jetpack docker up restart all Docker containers. Consider changing to "Docker containers" (plural) for accuracy.

Suggested change
**Note:** Be sure to restart the Docker container after making the changes:
**Note:** Be sure to restart the Docker containers after making the changes:

Copilot uses AI. Check for mistakes.
Copy link
Contributor

Choose a reason for hiding this comment

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

It's one parent container, so singular is fine.

…tion

Refine the instructions for resolving cURL error 35 when connecting Jetpack to WordPress.com. Change "Solution" to "Method" for clarity and enhance the steps for configuring Docker Desktop proxy settings and wp-config.php. Include a reminder to restart Docker containers after applying changes.
Copy link
Member

@jeherve jeherve left a comment

Choose a reason for hiding this comment

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

Thanks for adding this in. This may be an opportunity to update and consolidate the Jurassic Tube instructions in this repo:

For a while, I was considering adding the 2 constants to everyone's wp-config.php file, but that does cause some issues in some environments (folks not using Autoproxxy, or folks working on Jetpack Boost development and working with a Boost Cloud dev environment next to the Jetpack environment) so I'm afraid it cannot be something we enable for everyone.

tbradsha
tbradsha previously approved these changes Nov 12, 2025
Copy link
Contributor

@tbradsha tbradsha left a comment

Choose a reason for hiding this comment

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

One inline comment, but no blockers. Thanks!

Applying suggested edit

Co-authored-by: tbradsha <32492176+tbradsha@users.noreply.github.com>
- Remove deprecated Docker-included Jurassic Tube instructions
- Link to standalone Jurassic Tube setup instructions (PCYsg-GJ2-p2)
- Reframe proxy configuration as recommended settings rather than just troubleshooting
- Add note about when proxy settings may cause issues (non-Autoproxxy users, Boost developers)
- Make tools/docker/README.md the comprehensive Jurassic Tube guide
- Shorten docs/quick-start.md section and link to detailed Docker documentation
- Improves clarity and reduces documentation duplication
@DevinWalker
Copy link
Author

@jeherve Thanks for the feedback! I've updated the PR to consolidate the Jurassic Tube documentation:

Changes made:

  • Removed the deprecated Docker-included Jurassic Tube instructions from tools/docker/README.md
  • Updated to link to the standalone version (PCYsg-GJ2-p2) instead
  • Made tools/docker/README.md the comprehensive guide with detailed proxy configuration
  • Shortened the docs/quick-start.md section and added a link to the detailed Docker documentation
  • Reframed proxy settings as recommended configuration (with a note about Autoproxxy/Boost compatibility issues)

This should eliminate the duplication and make tools/docker/README.md the single source of truth for Jurassic Tube with Docker. Let me know if you'd like any other adjustments!

@DevinWalker DevinWalker added [Status] Needs Review This PR is ready for review. and removed [Status] Needs Author Reply We need more details from you. This label will be auto-added until the PR meets all requirements. [Status] In Progress labels Nov 13, 2025
@DevinWalker DevinWalker requested a review from jeherve November 13, 2025 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants