Skip to content

Update nextjs in webhook example#676

Merged
lukasIO merged 2 commits into
mainfrom
lukas/update-next-js
Jun 10, 2026
Merged

Update nextjs in webhook example#676
lukasIO merged 2 commits into
mainfrom
lukas/update-next-js

Conversation

@lukasIO

@lukasIO lukasIO commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review


export async function POST(req: Request) {
const body = await req.text();
const event = await receiver.receive(body, req.headers.get('Authorization') ?? undefined);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚩 SDK exports misleading authorizeHeader = 'Authorize' constant

The SDK at packages/livekit-server-sdk/src/WebhookReceiver.ts:9 exports authorizeHeader = 'Authorize', but the actual header used by LiveKit server is Authorization (per the SDK README examples at lines 140, 173 which use req.get('Authorization')). The new example correctly uses req.headers.get('Authorization'). The exported constant is unused within the SDK and appears to be a pre-existing inconsistency — it's not consumed anywhere in the codebase.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

@lukasIO lukasIO merged commit 801d4fd into main Jun 10, 2026
12 checks passed
@lukasIO lukasIO deleted the lukas/update-next-js branch June 10, 2026 14:02
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.

2 participants