Skip to content

Add pubsub system for websocket server #29719

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
Draft
wants to merge 79 commits into
base: main
Choose a base branch
from
Draft

Add pubsub system for websocket server #29719

wants to merge 79 commits into from

Conversation

anbraten
Copy link
Contributor

@anbraten anbraten commented Mar 11, 2024 •
edited
Loading

As requested in #28958 this PR tries to add an interface and memory pubsub system for the websocket server to be able to use a pubsub system like redis later on.

This PR contains changes from #28958.

image

anbraten and others added 30 commits January 27, 2024 14:54
@anbraten
Update issue comments
2b247dd
@anbraten
move dep
5005245
@anbraten
fix htmx ws loading
7cf50ef
@anbraten
Merge remote-tracking branch 'upstream/main' into issue-updates
4947abc
@anbraten
add listeners
3b35637
@anbraten
Merge branch 'main' into issue-updates
e85f85b
@anbraten
add websocket service
d77073b
@anbraten
improve websocket service
17f4faf
@anbraten
cleanup
948f1bd
@anbraten
update templates
1d6a34d
@anbraten
Merge branch 'main' into issue-updates
337456a
@anbraten
Merge remote-tracking branch 'upstream/main' into issue-updates
e4beba8
@anbraten
adust template
21bcb1b
@anbraten
Merge branch 'issue-updates' of github.com:anbraten/gitea into issue-…
c2ca7d8 
…updates
@anbraten
undo some changes
fb49e6b
@anbraten
sort imports
4c03093
@anbraten
render comment template
7173bea
@anbraten
improve session data and rendering
bc2a352
@anbraten
Merge remote-tracking branch 'upstream/main' into issue-updates
26ab35c
@anbraten
cleanup
b2f4aae
@anbraten
Merge branch 'main' into issue-updates
afcdd78
@anbraten
Merge remote-tracking branch 'upstream/main' into issue-updates
21485f8
@anbraten
Merge branch 'issue-updates' of github.com:anbraten/gitea into issue-…
7cda3aa 
…updates
@anbraten
Merge branch 'main' into issue-updates
d9c4554
@anbraten
cleanup
ea06a25
@anbraten
Merge remote-tracking branch 'upstream/main' into issue-updates
9ace44c
@anbraten
finish
78b94ff
@anbraten
fix permission
c595916
@anbraten
sort imports
b805631
@anbraten
Merge branch 'main' into issue-updates
e9ded26
@anbraten
Copy link
Contributor Author

@anbraten is there a relation/dependency between this and #28958? Should one land first?

This is meant to be the follow-up PR for #28958 (that one should go in first) adding the mutli-node support request by lunny

@anbraten anbraten mentioned this pull request Apr 3, 2024
Closed
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@silverwind silverwind silverwind left review comments

@techknowlogick techknowlogick techknowlogick left review comments

Assignees
No one assigned
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/dependencies modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@anbraten @silverwind @techknowlogick @GiteaBot