Skip to content

Conversation

@minorun365
Copy link
Contributor

Description

This PR adds environment variable configuration (GDRIVE_CREDENTIALS_PATH) to the NPX setup for the Google Drive server, allowing users to properly specify the path to their credentials file.

Server Details

  • Server: gdrive
  • Changes to: configuration examples (added GDRIVE_CREDENTIALS_PATH to NPX setup)

Motivation and Context

This PR addresses the issue where the credentials path wasn't explicitly specified in the NPX configuration. By providing clear examples in the README for how to set up after authentication, this reduces setup errors and improves user experience.

How Has This Been Tested?

The configuration changes have been tested with Claude Desktop and Amazon Q Developer CLI when running the Google Drive server via NPX. It correctly reads credentials from the specified path after the authentication process is completed.

Breaking Changes

Existing users don't need to update their configurations for this change, but explicitly specifying the credentials file path will make their setup more reliable.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Protocol Documentation
  • My changes follows MCP security best practices
  • I have updated the server's README accordingly
  • I have tested this with an LLM client
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have documented all environment variables and configuration options

@olaservo olaservo added server-gdrive documentation Improvements or additions to documentation labels May 7, 2025
Copy link
Member

@olaservo olaservo left a comment

Choose a reason for hiding this comment

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

Thank you!

@olaservo olaservo merged commit bdf4323 into modelcontextprotocol:main May 7, 2025
3 checks passed
@minorun365 minorun365 deleted the patch-1 branch May 7, 2025 03:21
PazerOP referenced this pull request in PazerOP/mcp-template Jul 15, 2025
Fixed README: Added NPX config for Claude Desktop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants