avery42/delfin
9
146
Fork
You've already forked delfin
28

Add server discovery via UDP broadcast, with 50ms timeout #137

Open
selfhoster1312 wants to merge 1 commit from selfhoster1312/delfin:server-discovery into main
pull from: selfhoster1312/delfin:server-discovery
merge into: avery42:main
avery42:main
avery42:weblate-delfin-shared
avery42:quick-connect-check-enabled
avery42:132-deinterlacing
avery42:gst
avery42:windows
Contributor
Copy link

I wasn't sure if i should use tokio or sync networking, or glib event loop... I went with the standard library with a timeout of 50ms. Let me know if there is a better way to achieve the goal.

Tested and it works!

I wasn't sure if i should use tokio or sync networking, or glib event loop... I went with the standard library with a timeout of 50ms. Let me know if there is a better way to achieve the goal. Tested and it works!
selfhoster1312 force-pushed server-discovery from a75973a4fe
Some checks failed
ci/woodpecker/pr/release Pipeline was successful
ci/woodpecker/pr/test Pipeline was successful
ci/woodpecker/pr/lint Pipeline failed
to 8b460d19a0
All checks were successful
ci/woodpecker/pr/release Pipeline was successful
ci/woodpecker/pr/test Pipeline was successful
ci/woodpecker/pr/lint Pipeline was successful
2024年08月18日 19:28:37 +02:00 Compare
avery42 force-pushed server-discovery from 8b460d19a0
All checks were successful
ci/woodpecker/pr/release Pipeline was successful
ci/woodpecker/pr/test Pipeline was successful
ci/woodpecker/pr/lint Pipeline was successful
to 15abff66a4
All checks were successful
ci/woodpecker/pr/release Pipeline was successful
ci/woodpecker/pr/test Pipeline was successful
ci/woodpecker/pr/lint Pipeline was successful
2024年10月06日 19:45:44 +02:00 Compare
All checks were successful
ci/woodpecker/pr/release Pipeline was successful
ci/woodpecker/pr/test Pipeline was successful
Required
Details
ci/woodpecker/pr/lint Pipeline was successful
Required
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u server-discovery:selfhoster1312-server-discovery
git switch selfhoster1312-server-discovery
Sign in to join this conversation.
No reviewers
Labels
Clear labels
Kind/Breaking
Breaking change that won't be backward compatible
Kind/Bug
Something is not working
Kind/Documentation
Documentation changes
Kind/Enhancement
Improve existing functionality
Kind/Feature
New functionality
Kind/Security
This is security issue
Kind/Testing
Issue or pull request related to testing
Needs Design
Packaging/AUR
Issue with the AUR package
Packaging/Flatpak
Issue with the Flatpak
Priority
Critical
 The priority is critical
Priority
High
 The priority is high
Priority
Low
 The priority is low
Priority
Medium
 The priority is medium
Reviewed
Confirmed
 Issue has been confirmed
Reviewed
Duplicate
 This issue or pull request already exists
Reviewed
Invalid
 Invalid issue
Reviewed
Upstream
 Issue with a package or library we use that can't be fixed independently
Reviewed
Won't Fix
 This issue won't be fixed
Status
Abandoned
 Somebody has started to work on this but abandoned work
Status
Blocked
 Something is blocking this issue or pull request
Status
Need More Info
 Feedback is required to reproduce issue or to continue work
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
avery42/delfin!137
Reference in a new issue
avery42/delfin
No description provided.
Delete branch "selfhoster1312/delfin:server-discovery"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?