Skip to content

dedupe notification auth headers#221

Open
saurabhhhcodes wants to merge 1 commit into
chthonn:mainfrom
saurabhhhcodes:codex/notification-auth-header-helper
Open

dedupe notification auth headers#221
saurabhhhcodes wants to merge 1 commit into
chthonn:mainfrom
saurabhhhcodes:codex/notification-auth-header-helper

Conversation

@saurabhhhcodes

Copy link
Copy Markdown

Summary

  • Extract a shared JSON auth header helper for notification requests
  • Omit x-auth-token entirely when no token exists, so the server no longer sees a literal "null" header value
  • Remove the duplicated header object from the four calls in NotificationListener

Validation

  • git diff --check
  • npm run build in frontend

@vercel

vercel Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Someone is attempting to deploy a commit to the Sunil Kumar's projects Team on Vercel.

A member of the Team first needs to authorize it.

@saurabhhhcodes saurabhhhcodes changed the title [codex] dedupe notification auth headers dedupe notification auth headers Jun 13, 2026
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.

1 participant