Commit Graph

5 Commits (619e5fe16dd20045050827b8a18febefb6151edf)

Author SHA1 Message Date
Guewen Baconnier 619e5fe16d Prevent to send web notifications to other users
Only the admin user (sudo) is allowed to send notifications to other
users. The normal users can only send notifications to themselves.

This is to prevent attackers to craft malicious notifications and send
them to other users using RPC.

Correction based on the idea of @hbrunn
2024-10-14 08:53:21 +07:00
Damien Bouvy 687baa82b4 [MIG] web_notify: Migration to 11.0
- Use the 'session' class of the JS Framework (session no lounger bound
to web client)
- Test change: compare emitted & received messages based on content, not
order. Using string comparison raises false positives.
2024-10-14 08:53:21 +07:00
Houzéfa Abbasbhay ed2e6f0ea2 [10.0][FIX] web_notify tests: Fix an arg check
Fix a check when comparing a user count with items within a mock call.

The previous method was succeeding by pure luck because OCA test
databases contain 2 users, which happens to be the amount of items
within a mock "call_args" (it contains args + kwargs).
2024-10-14 08:53:21 +07:00
Jay Vora(SerpentCS) 2be1034f95 [MIG] Migration started web_notify 2024-10-14 08:53:21 +07:00
Laurent Mignon (ACSONE) 7b6096266f New module web_notify
This technical module allows you to send instant notification messages from the server to the user in live.
2024-10-14 08:53:21 +07:00