Skip to content

Conversation

@stevengonsalvez
Copy link
Contributor

@stevengonsalvez stevengonsalvez commented Feb 6, 2025

Description

Fix for issue #556

  • added logging
  • added fetch package

Server Details

  • Server: github
  • Changes to: tools -> create_issue

Motivation and Context

solves issue raised in #556

How Has This Been Tested?

  • inspector
  • tested with claude desktop

Breaking Changes

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

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

- Improve error handling in GitHub issue creation process
- Add detailed error messages for repository not found scenarios
- Implement global fetch polyfill for node-fetch
- Add debug and error logging for issue creation attempts
@stevengonsalvez stevengonsalvez changed the title Add error handling and logging for GitHub issue creation Added fetch package #556 Feb 6, 2025
@olaservo olaservo added server-github bug Something isn't working labels Mar 27, 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.

Thanks for fixing this and for your patience in this getting merged.

@olaservo olaservo merged commit 94a29d9 into modelcontextprotocol:main Mar 29, 2025
7 checks passed
PazerOP referenced this pull request in PazerOP/mcp-template Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants