Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add push notification daemon for proper handling of push notifications #387

Merged
merged 51 commits into from
Nov 28, 2024
Merged

Add push notification daemon for proper handling of push notifications #387

merged 51 commits into from
Nov 28, 2024

Conversation

IAmTomahawkx
Copy link
Member

This PR adds pushd, the new push notification daemon. It also adds rabbitmq as a required component for running the backend, which will be considered a breaking change for selfhosting (as it modifies existing config keys).

Please make sure to check the following tasks before opening and submitting a PR

  • I understand and have followed the contribution guide
  • I have tested my changes locally and they are working as intended

@IAmTomahawkx
feat: create base of push daemon
e510757 
Signed-off-by: IAmTomahawkx <[email protected]>
@IAmTomahawkx
Add outbound senders
299d0f6
@IAmTomahawkx
Make web_push send to rabbit instead (temp stuff)
bc96bfb
@IAmTomahawkx
feat: stability and friend requests
71583de
@IAmTomahawkx
make vapid fr stuff not suck
07b31a9
@IAmTomahawkx
swap naming of queue
79d333f
@IAmTomahawkx
Merge branch 'main' of https://github.com/revoltchat/backend into fea…
0510295 
…t/pushd
@IAmTomahawkx
move pushd into daemons folder
3b435e8
@IAmTomahawkx
fix cargo file for move into daemons folder
2c78765
@IAmTomahawkx
feat: probably working fcm push notifs
263d16b
@IAmTomahawkx
comment out fcm webpush stuff since the config keys dont exist
22dedea
@IAmTomahawkx
fix fcm, name queues according to their prod status and configure rou…
449db24 
…ting keys
@IAmTomahawkx
add pushd to docker
7d6ed7a
@IAmTomahawkx
mix: Remove old code, add stuff to pushd
68324b1
@IAmTomahawkx
Merge branch 'main' of https://github.com/revoltchat/backend into fea…
d19a9c9 
…t/pushd
@IAmTomahawkx
fix: lockfile
25f8f13
@IAmTomahawkx
feat: update rocket to 5.0.1
2ccb408
@IAmTomahawkx
fix: fix queues and ack bugs
bc641c3
@IAmTomahawkx
Move rabbit messsage processing into ack queue
e3b0ebb
@IAmTomahawkx
chore: update readme
8672388
@IAmTomahawkx
Merge branch 'main' of https://github.com/revoltchat/backend into fea…
48735e9 
…t/pushd
@IAmTomahawkx
Merge branch 'main' of https://github.com/revoltchat/backend into fea…
5a8c556 
…t/pushd
@IAmTomahawkx
chore: optimizations for ack database hits
41fdb0d
@IAmTomahawkx
pushd flowchart
0340f65
@IAmTomahawkx
misc: update flowchart
b9e6843
@IAmTomahawkx
exit dependancy hell
f128683
@IAmTomahawkx
add rocket_impl flag to authifier
f6851ea
@IAmTomahawkx
make the tests file of delta actually compile
424f983
@IAmTomahawkx
fix: don't silence every push message
795e6af
@IAmTomahawkx
fix: don't silence all messages
c16da74
@IAmTomahawkx
add debug logging for sending data to rabbit from message events
346bc01
@IAmTomahawkx
validate mentions at a server membership level
77bb7db
@IAmTomahawkx
put back that import that was actually important
323fb60
@IAmTomahawkx
Merge branch 'main' into feat/pushd
6cccd0b
@IAmTomahawkx
minor fix to lockfile
33d7376
@IAmTomahawkx
update delta authifier
0954fb2
@IAmTomahawkx
feat: proper permissions for push notifications
0fda34c
@IAmTomahawkx
add unit test for mention sanitization
5377294
@IAmTomahawkx
remove local file dependancy on authifier
7d8bad4
@IAmTomahawkx
Merge branch 'main' into feat/pushd
2c6e170
@IAmTomahawkx
update ports to proper defaults
9ea5da8
@IAmTomahawkx
fixTM the node bindings
0643a38
@IAmTomahawkx
Theoretically configure docker releases for pushd
70274bf 
Signed-off-by: IAmTomahawkx <[email protected]>
@IAmTomahawkx
declare exchange in pushd and delta
2f8506f
@IAmTomahawkx
fix createbuckets script
c2a0efe 
Signed-off-by: IAmTomahawkx <[email protected]>
@IAmTomahawkx
fix: reference db implementation
77aafd0 
Signed-off-by: IAmTomahawkx <[email protected]>
@IAmTomahawkx
fix: remove finally redundant code
7b73fc3 
Signed-off-by: IAmTomahawkx <[email protected]>
insertish
insertish requested changes Nov 27, 2024
View reviewed changes
README.md Show resolved Hide resolved
Cargo.toml Outdated Show resolved Hide resolved
compose.yml Show resolved Hide resolved
crates/core/database/src/models/channel_unreads/ops/mongodb.rs Show resolved Hide resolved
crates/core/database/src/models/server_members/ops/reference.rs Show resolved Hide resolved
crates/core/models/src/v0/channels.rs Outdated Show resolved Hide resolved
crates/daemons/pushd/src/consumers/base.rs Outdated Show resolved Hide resolved
crates/daemons/pushd/src/consumers/inbound/ack.rs Outdated Show resolved Hide resolved
crates/daemons/pushd/src/consumers/inbound/internal.rs Outdated Show resolved Hide resolved
crates/daemons/pushd/src/consumers/outbound/vapid.rs Outdated Show resolved Hide resolved
@IAmTomahawkx IAmTomahawkx merged commit b55765d into main Nov 28, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@insertish insertish insertish approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Pull Request Overview
Status: ✅ Merged
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@IAmTomahawkx @insertish