Create push notification config returns no id on database-backed stores

オープン 初心者向け
#1,237 コメント 1 件 リアクション 0 件 担当者 1 名 GitHub で見る

@rohityan がすでに取り組んでいます。

2026年9月15日 から。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
75/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
領域
backend, databases

調査の方向性

この issue は default_request_handler.py の 556 行目と default_request_handler_v2.py の 394 行目にあり、on_create_task_push_notification_config が永続化されたオブジェクトではなく request の params を返しています。inmemory_push_notification_config_store.py:51 と database_push_notification_config_store.py:292 を比較して、それぞれの store が id をどのように扱っているか確認してください。database store を使用して push notification config を作成し、返された config に有効な id があることを検証するテストを書いてください。既存の push notification テストを実行して、修正によって他のものが壊れないことを確認してください。

索引モデルが issue の本文から書いたものです。

説明

component: server status:awaiting response

What happens

Creating a push notification config against a database-backed server returns a config with no id. Reading it back with that id raises InvalidParamsError.

create returned id = ''
get FAILED -> InvalidParamsError: Validation failed

The same calls against the in-memory store return id='task-123' and the get succeeds.

Why

on_create_task_push_notification_config returns the caller's request object rather than what the store persisted (default_request_handler.py:556, default_request_handler_v2.py:394).

Both stores default an empty id to the task id, on different objects:

  • inmemory_push_notification_config_store.py:51 sets it on the caller's object, so return params carries it.
  • database_push_notification_config_store.py:292 copies first and sets it on the copy, so params never gets it.

The store holds the right id either way; only the response differs.

Version

main @ d55a3d3

主要言語
Python
スター
2.2k
フォーク
496
平均マージ
1日 23時間
マージ済み PR（30日）
16

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

a2aproject/a2a-python のほかの issue

a2aproject/a2a-python の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。