matrix-org / matrix-org/matrix-hookshot
Static webhooks
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 450
- Forks
- 95
- Avg merge
- 1d 2h
- Merged PRs (30d)
- 23
Description
Statically configured webhooks are not functioning as expected.
While checking the codebase, I noticed that the method GenericHookConnection.createConnectionForState contains the logic and validation necessary to instantiate a GenericHookConnection with the isStatic = true flag. However, this method is never actually invoked or utilized across the application.
Although the documentation explicitly states that static webhooks are supported, they are practically broken and do not work in the current implementation. What is the real situation with this feature?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing GenericHookConnection.createConnectionForState and searching the application for its callers, then compare that behavior with the documentation's static webhook claims. Determine whether static webhooks are intended to work and identify the missing integration or documentation correction. Done means the feature's supported status is accurate and static webhook behavior is either functional or clearly documented as unsupported.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- api, backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100