Skip to content

Conversation

@ShireenMissi
Copy link
Contributor

Summary

Fixes an issue where Telegram “Send message and wait for response” custom forms remain stuck in a loading state after submission.

The backend correctly receives and processes the form data, but the frontend fails due to missing CORS headers on waiting webhook responses (especially when the request origin is "null").

This change ensures proper CORS headers are always sent for waiting webhooks, allowing the form to complete and render the submission confirmation page.

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/NODE-4141/community-issue-telegram-node-custom-form-passes-data-but-still-loads

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

Co-authored-by: Shireen Missi <[email protected]>
Co-authored-by: Michael Kret <[email protected]>
Co-authored-by: Michael Kret <[email protected]>
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 5 files

@codecov
Copy link

codecov bot commented Jan 5, 2026

Codecov Report

❌ Patch coverage is 85.71429% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
packages/cli/src/webhooks/waiting-forms.ts 66.66% 1 Missing ⚠️
packages/cli/src/webhooks/waiting-webhooks.ts 50.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Jan 5, 2026
@blacksmith-sh

This comment has been minimized.

@currents-bot
Copy link

currents-bot bot commented Jan 5, 2026

E2E Tests: n8n tests passed after 6m 53.6s

🟢 592 · 🔴 0 · ⚪️ 12 · 🟣 3

View Run Details

Run Details

  • Project: n8n

  • Groups: 2

  • Framework: Playwright

  • Run Status: Passed

  • Commit: b6afcb1

  • Spec files: 99

  • Overall tests: 604

  • Duration: 6m 53.6s

  • Parallelization: 9

Groups

GroupId Results Spec Files Progress
ui 🟢 541 · 🔴 0 · ⚪️ 12 · 🟣 3 92 / 92
ui:isolated 🟢 51 · 🔴 0 · ⚪️ 0 7 / 7


This message was posted automatically by currents.dev | Integration Settings

@RomanDavydchuk RomanDavydchuk merged commit 8ea741a into 1.x Jan 5, 2026
61 of 63 checks passed
@RomanDavydchuk RomanDavydchuk deleted the nodes-4141-into-1x branch January 5, 2026 14:41
@n8n-assistant n8n-assistant bot mentioned this pull request Jan 8, 2026
@n8n-assistant
Copy link
Contributor

n8n-assistant bot commented Jan 8, 2026

Got released with [email protected]

@gpundt456
Copy link

Any idea on when this fix will be available?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team Released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants