HTTP Based Master Server + Multithreading
LJ's gonna come in here and say I fucked up the mutexes, huh?
There are 3 reasons for you to review and merge this...
- HTTP means connections go through Cloudfare, so the address of mb.srb2.org isn't public.
- No memory leaks from the decade old Master Server daemon.
- Multithreading means the game does not hang if the Master Server is slow.
Also check out the public REST API.
-
MasterBased? Based on what?
-
Owner
Based on php. Oh, yikes...
-
changed the description
Toggle commit list -
-
src/hms123311.c 0 → 100644
587 588 hms = HMS_connect("versions/%d", MODID); 589 590 if (! hms) 591 return 0; 592 593 ok = 0; 594 595 if (HMS_do(hms)) 596 { 597 version = strtok(hms->buffer, " "); 598 version_name = strtok(0, "\n"); 599 600 if (version && version_name) 601 { 602 if (atoi(version) != MODVERSION) -
Owner
I think this is worth pointing out specifically. This is intentionally comparing the exact value rather than greater than because it makes it very obvious if we forget to update MODVERSION on the MSCP every single release.
-
-
-
changed target branch from
mastertonextToggle commit list -
changed the description
Toggle commit list -
-
-
Hopefully, this makes it into 2.2.5, because this really improves the performance of SRB2.
-
marked as a Work In Progress
Toggle commit list -
-
-
closed
Toggle commit list -
reopened
Toggle commit list -
unmarked as a Work In Progress
Toggle commit list -
added 44 commits
-
479bbeac...00ceca8c - 42 commits from branch
next - 93615b07 - Merge remote-tracking branch 'origin/next' into http-mserv
- f5c5ec00 - Use SRB2VERSION here...
Toggle commit list -
479bbeac...00ceca8c - 42 commits from branch
-
added 201 commits
-
f5c5ec00...cc677d2d - 200 commits from branch
next - 54e44619 - Merge remote-tracking branch 'origin/next' into http-mserv
Toggle commit list -
f5c5ec00...cc677d2d - 200 commits from branch
-
Owner
This has been in regular usage in the Kart community for the past three months. There may be some edge case bugs, and some have reported issues with Cloudflare, but that is a different issue.
-
merged
Toggle commit list -
