Skip to content

macOS server support - #691

Merged
RDIL merged 6 commits into
masterfrom
reece/macos-server
May 2, 2026
Merged

macOS server support#691
RDIL merged 6 commits into
masterfrom
reece/macos-server

Conversation

@RDIL

@RDIL RDIL commented May 2, 2026

Copy link
Copy Markdown
Member

Scope

Several patches to let the game log in to the server via the macOS steam version.

The steam version works, the apple version doesn't yet.

Test Plan

Log in successfully from macOS Steam version of the game with --disable-dynamic-resources.

Checklist


General

  • I've run Prettier to format any changed files
  • I've verified that my changes work, and included a test plan

Testing

  • I have added or considered adding unit/integration tests that cover any code changes

Comment thread components/index.ts

app.use(function badPathRewritingMiddleware(req, _, next) {
req.url = req.url.replaceAll("//", "/")
// rewrite every `//` to `/` that occurs before the query string

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?issuer=https://appleid.apple.com, because that makes total sense

@RDIL
RDIL marked this pull request as ready for review May 2, 2026 19:43
Comment thread patcher-darwin/src/main.cpp Outdated

@alex73630 alex73630 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

(RIP Stadia o7)

@alex73630

Copy link
Copy Markdown
Contributor

Just a quick question, have we tested against Intel Macs or those are not supported by the game anyway?

@AnthonyFuller

Copy link
Copy Markdown
Contributor

Not supported.

@RDIL
RDIL merged commit c94fcd7 into master May 2, 2026
5 checks passed
@RDIL
RDIL deleted the reece/macos-server branch May 2, 2026 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants