Skip to content

repair doc strings for request method - #2558

Merged
Sehat1137 merged 1 commit into
ag2ai:mainfrom
HelgeKrueger:patch-1
Oct 8, 2025
Merged

Sehat1137 merged 1 commit into
ag2ai:mainfrom
HelgeKrueger:patch-1

Conversation

@HelgeKrueger

@HelgeKrueger HelgeKrueger commented Oct 8, 2025 •

Copy link
Copy Markdown
Contributor

First, I adjusted the docstring to match the one of RabbitBroker. It makes things a bit less confusion.

Second, I was looking at the code to figure out if TestRabbitBroker could also set the "reply-to" property of the incoming message. Currently, when using with the fake broker, one gets when evaluating Context('message'):

RabbitMessage(raw_message=PatchedMessage:{'app_id': 'faststream-0.5.48',
...
 'reply_to': None,

whereas with the real broker, one gets

RabbitMessage(raw_message=IncomingMessage:{'app_id': 'faststream-0.5.48',
...
 'reply_to': 'amq.rabbitmq.reply-to.g1h2AA5yZXBseUA3NzI4NzE3MgAAUkkAAAAAaOPRjw==.ms5TOURI+Qs2Y1biv9El6w==',

@CLAassistant

CLAassistant commented Oct 8, 2025 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@Sehat1137
Sehat1137 enabled auto-merge October 8, 2025 10:23
@Sehat1137
Sehat1137 added this pull request to the merge queue Oct 8, 2025
Merged via the queue into ag2ai:main with commit 935b7de Oct 8, 2025
32 checks passed
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.

3 participants