Skip to content

Fix hang on main handler registration, cleanup sample handler#115

Merged
kzu merged 3 commits into
mainfrom
dev/handlerhang
Jun 10, 2025
Merged

Fix hang on main handler registration, cleanup sample handler#115
kzu merged 3 commits into
mainfrom
dev/handlerhang

Conversation

@kzu
Copy link
Copy Markdown
Member

@kzu kzu commented Jun 10, 2025

No description provided.

kzu and others added 3 commits June 10, 2025 03:05
Unlike the lambda/anonymous function handler, the class one never resolves and hangs the process.
…pHandler

IWhatsAppHandler is already registered by the builder to the the entire pipeline.

So, we should try to resolve the provided concrete handler type and resolve all the dependencies defined in the ctor.
@kzu kzu added the bug Something isn't working label Jun 10, 2025
@kzu kzu enabled auto-merge (squash) June 10, 2025 13:17
@kzu
Copy link
Copy Markdown
Member Author

kzu commented Jun 10, 2025

32 passed 32 passed 7 skipped

🧪 Details on Ubuntu 24.04.2 LTS

from dotnet-retest v0.7.1 on .NET 8.0.16 with 💜 by @devlooped

@kzu kzu merged commit 7d950f0 into main Jun 10, 2025
7 checks passed
@kzu kzu deleted the dev/handlerhang branch June 10, 2025 13:18
@devlooped devlooped locked and limited conversation to collaborators Jul 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants