Skip to content

ci: fix private key processing in MCP login#53

Merged
masnwilliams merged 2 commits intomainfrom
mason/ga-release
Sep 10, 2025
Merged

ci: fix private key processing in MCP login#53
masnwilliams merged 2 commits intomainfrom
mason/ga-release

Conversation

@masnwilliams
Copy link
Copy Markdown
Collaborator

@masnwilliams masnwilliams commented Sep 10, 2025

TL;DR

Fixed the publish-mcp CI workflow by correctly parsing the multi-line private key secret required for login.

Why we made these changes

The publish-mcp workflow was failing at the login step because the private key, stored as a GitHub secret, was not being processed correctly. The script couldn't handle the multi-line format of the key, which blocked automated publishing to the MCP registry.

What changed?

  • In .github/workflows/publish-mcp.yml, the 'Login to MCP Registry' step was updated.
  • The script now explicitly reconstructs the PEM-formatted private key from the secret.
  • It strips extra whitespace and newlines before using openssl to extract the key material, making the process robust against formatting issues in the secret.

Description generated by Mesa. Update settings

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
mcp Ready Ready Preview Comment Sep 10, 2025 10:22pm

@masnwilliams masnwilliams merged commit 9b08071 into main Sep 10, 2025
1 of 2 checks passed
@masnwilliams masnwilliams deleted the mason/ga-release branch September 10, 2025 22:20
Copy link
Copy Markdown

@mesa-dot-dev mesa-dot-dev Bot left a comment

Choose a reason for hiding this comment

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

Performed full review of 671c4bb...367fc6e

Tip

⚡ Quick Actions

This review was generated by Mesa.

Actions:

Slash Commands:

  • /review - Request a full code review
  • /review latest - Review only changes since the last review
  • /describe - Generate PR description. This will update the PR body or issue comment depending on your configuration
  • /help - Get help with Mesa commands and configuration options

1 files reviewed | 0 comments | Review on Mesa | Edit Reviewer Settings

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.

1 participant