Skip to content

Feature/media proxy pass#531

Open
daviddarke wants to merge 40 commits into
pre-releasefrom
feature/media-proxy-pass
Open

Feature/media proxy pass#531
daviddarke wants to merge 40 commits into
pre-releasefrom
feature/media-proxy-pass

Conversation

@daviddarke

Copy link
Copy Markdown
Member

Media Proxy command added. The ability to pass through the HTTP auth that's enabled by default on our staging sites is essential. Obviously, those creds need to be accessible, so the standard .env variables have been updated:

SETUP_INSTALL_AND_BUILD_ONLY=false
WORDPRESS_AUTOMATION_USER="Bot"
WORDPRESS_AUTOMATION_PASSWORD="password"
WORDPRESS_USER=""
WORDPRESS_USER_EMAIL=""
WORDPRESS_PASSWORD=""

STAGING_SSH_USERNAME=""
STAGING_SSH_HOST=""
STAGING_SSH_PORT=""
STAGING_URL="//stg-staging.kinsta.cloud"
STAGING_HTTP_AUTH_USERNAME=""
STAGING_HTTP_AUTH_PASSWORD=""

STAGING_HTTP_AUTH_USERNAME and STAGING_HTTP_AUTH_PASSWORD are new additions.

If these variables aren't preset, no Auth header is added to the proxy.

@changeset-bot

changeset-bot Bot commented Apr 8, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9d9e79e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@atomicsmash/cli Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@daviddarke

daviddarke commented Apr 8, 2026

Copy link
Copy Markdown
Member Author

Also, I wrote most of this, but Claude really helped with the "closing brace" detection logic.

@mikeybinns mikeybinns left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not pulled down for testing yet, but found some minor static code changes

Comment thread packages/cli/src/commands/toggle-media-proxy.ts Outdated
Comment thread packages/cli/src/commands/toggle-media-proxy.ts Outdated
Comment thread packages/cli/src/commands/toggle-media-proxy.ts Outdated
Comment thread packages/cli/src/commands/toggle-media-proxy.ts Outdated
Comment thread packages/cli/src/commands/toggle-media-proxy.ts Outdated
Comment thread packages/cli/src/commands/toggle-media-proxy.ts Outdated
Comment thread packages/cli/src/commands/toggle-media-proxy.ts Outdated
Comment thread .changeset/lucky-games-pump.md
Base automatically changed from pre-release to main May 22, 2026 09:55
@mikeybinns mikeybinns changed the base branch from main to pre-release June 10, 2026 14:54
@mikeybinns mikeybinns self-requested a review June 12, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants