Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

get_discussion and get_discussion_comments accept calls with missing required parameters instead of returning a validation error

Aperta Adatta ai principianti
#2,740 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
76/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Tranquilla
Stack tecnologico
go
Ambito
api, backend-api-design

Direzione di ricerca

Inizia individuando gli handler get_discussion e get_discussion_comments e confronta la loro decodifica dei parametri con quella degli handler di scrittura di Discussion convalidati e aggiornati in PR #2718. Riproduci chiamate omettendo ciascun campo obbligatorio, quindi conferma che entrambi gli strumenti di lettura restituiscano un errore di validazione denominato per parametro mancante senza inviare una richiesta GraphQL.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Describe the bug

When get_discussion or get_discussion_comments is called without one of the required parameters (owner, repo, or discussionNumber), the tool does not return a parameter-validation error. Instead it silently substitutes a zero value and issues a GraphQL request, which returns a confusing API-level error rather than a clear message identifying the missing field.

Affected version: current (tested against ghcr.io/github/github-mcp-server latest as of 2026-06)

Steps to reproduce
  1. Configure the MCP server normally.
  2. Call get_discussion with only {"repo": "myrepo", "discussionNumber": 1} (omitting owner).
  3. Observe that the tool returns a GraphQL error like "Could not resolve to a Repository" rather than a message indicating owner is required.

Repeat with get_discussion_comments and any missing required field.

Expected behavior

The tool should return a structured error immediately: "missing required parameter: owner" (or equivalent), consistent with how add_discussion_comment and the other discussion write tools behave when a required parameter is absent.

Actual behavior

The tool issues a GraphQL query with the missing field set to its zero value ("" for strings, 0 for numbers) and returns whichever API error results from that invalid query.

Additional context

The write handlers in the same file (add_discussion_comment, reply_to_discussion_comment, etc.) were updated in PR #2718 to use explicit parameter validation that returns named errors on missing input. The two read handlers were not included in that pass and still use the legacy decoding pattern.

Lingua principale
Go
Stelle
33.1k
Fork
5k
Merge medio
2g 1h
PR unite (30g)
25

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di github/github-mcp-server

Tutte le issue di github/github-mcp-server

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.