-
Notifications
You must be signed in to change notification settings - Fork 252
feat: Add Redis-based workspace stream quota for WebRTC sessions #2025
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.
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
Merged
rafel-roboflow
merged 51 commits into
main
from
feat/dg-232-set-rate-limit-to-10-concurrent-streams-and-update
Mar 19, 2026
Merged
Changes from 47 commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
697b6e6
feat: Add Redis-based workspace stream quota for WebRTC sessions
rafel-roboflow 269d2c4
Merge branch 'main' into feat/dg-232-set-rate-limit-to-10-concurrent-…
rafel-roboflow b9cefda
added debug
rafel-roboflow e1af7f9
Merge branch 'main' of github.com:roboflow/inference into feat/dg-232…
rafel-roboflow b83f828
Merge branch 'feat/dg-232-set-rate-limit-to-10-concurrent-streams-and…
rafel-roboflow fd0c661
more debug; pass envs to modal env
rafel-roboflow b7c9a62
feat: add detailed Redis logging for session tracking debugging"
rafel-roboflow 3de18e7
Fix issue with RF-Detr model post-processing in TRT
PawelPeczek-Roboflow 4362ca0
Merge remote-tracking branch 'origin/fix/patch-post-processing-in-rfd…
rafel-roboflow ee50855
clean up
rafel-roboflow 37650bd
Merge branch 'main' into feat/dg-232-set-rate-limit-to-10-concurrent-…
rafel-roboflow e19d8ac
clean up
rafel-roboflow 400f71e
Merge branch 'main' of github.com:roboflow/inference into feat/dg-232…
rafel-roboflow 002e2a3
Merge branch 'feat/dg-232-set-rate-limit-to-10-concurrent-streams-and…
rafel-roboflow aba5acd
set max conn webrtc to 10
rafel-roboflow 71b3048
isorted
rafel-roboflow 459bbd5
Merge branch 'main' into feat/dg-232-set-rate-limit-to-10-concurrent-…
rafel-roboflow 35b5c62
Merge branch 'main' of github.com:roboflow/inference into feat/dg-232…
rafel-roboflow c7c9e91
fix bug heartbeat session without auth; fix bug allowing refresh non …
rafel-roboflow 47fa8f2
Merge branch 'feat/dg-232-set-rate-limit-to-10-concurrent-streams-and…
rafel-roboflow 8a173d7
removed unnecessary workspace id from refresh webrtc api
rafel-roboflow 91a1e06
calling hearbeat in watchdog just at the beginning insterad of waitin…
rafel-roboflow 8d790fb
implemented deregister session to avoid keeping sessions forever
rafel-roboflow cd8ebc3
Merge branch 'main' of github.com:roboflow/inference into feat/dg-232…
rafel-roboflow 49f7f15
fix wrong endpoint refactor
rafel-roboflow b23bd72
include always workspace id
rafel-roboflow 3579f5d
set ttl on redis zadd to avoid redis growing without control
rafel-roboflow 5f7889f
prevent calls if not api key to /usage/plan
rafel-roboflow bfd3f93
Revert "prevent calls if not api key to /usage/plan"
rafel-roboflow 5a00f91
Merge branch 'main' into feat/dg-232-set-rate-limit-to-10-concurrent-…
grzegorz-roboflow 1fc3b1e
Merge branch 'main' of github.com:roboflow/inference into feat/dg-232…
rafel-roboflow 68c9342
add missing @with_route_exceptions_async decorator to WebRTC session …
rafel-roboflow 6f66d68
Add proper error handling and request validation to WebRTC session en…
rafel-roboflow dfe6c22
clean up unused var
rafel-roboflow 54731ae
Merge branch 'feat/dg-232-set-rate-limit-to-10-concurrent-streams-and…
rafel-roboflow 12b50de
added redis total sessions count
rafel-roboflow 037206d
Merge branch 'main' into feat/dg-232-set-rate-limit-to-10-concurrent-…
rafel-roboflow 8f03e1d
fix: return proper HTTP error codes for WebRTC session auth failures
rafel-roboflow c68fa5c
Merge branch 'main' of github.com:roboflow/inference into feat/dg-232…
rafel-roboflow 5bbca17
Merge branch 'feat/dg-232-set-rate-limit-to-10-concurrent-streams-and…
rafel-roboflow fe59347
Merge branch 'main' into feat/dg-232-set-rate-limit-to-10-concurrent-…
rafel-roboflow 61d6582
Merge branch 'main' of github.com:roboflow/inference into feat/dg-232…
rafel-roboflow f2811b8
debug adding heartbeat debug
rafel-roboflow c3821bd
Merge branch 'main' of github.com:roboflow/inference into feat/dg-232…
rafel-roboflow a418923
Merge branch 'feat/dg-232-set-rate-limit-to-10-concurrent-streams-and…
rafel-roboflow d092189
revert debug log
rafel-roboflow 08e4d70
Merge branch 'main' of github.com:roboflow/inference into feat/dg-232…
rafel-roboflow 074c78d
Merge branch 'main' into feat/dg-232-set-rate-limit-to-10-concurrent-…
rafel-roboflow 940c004
raise http 500 error if no workspace id found
rafel-roboflow fd4b7e9
Merge branch 'main' of github.com:roboflow/inference into feat/dg-232…
rafel-roboflow 4f0476c
Merge branch 'feat/dg-232-set-rate-limit-to-10-concurrent-streams-and…
rafel-roboflow File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lets raise if workspace_id is empty / None