Skip to content

fix(api): preserve notification server fields#4116

Open
luresun1988-bot wants to merge 1 commit into
SecureBananaLabs:mainfrom
luresun1988-bot:fix-notification-server-fields
Open

fix(api): preserve notification server fields#4116
luresun1988-bot wants to merge 1 commit into
SecureBananaLabs:mainfrom
luresun1988-bot:fix-notification-server-fields

Conversation

@luresun1988-bot

Copy link
Copy Markdown

Fixes #4115

Summary

  • ignores caller-supplied id and read fields during notification creation
  • preserves the server-generated ntf_<timestamp> id
  • ensures new notifications always start with read: false
  • adds a regression test covering client-controlled field override attempts

Tests

  • node --test src/tests/*.test.js

github-actions Bot added a commit that referenced this pull request Jun 3, 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.

Notification creation should ignore client id and read fields

1 participant