Skip to content

Conversation

@xezon
Copy link

@xezon xezon commented Oct 13, 2025

Merge with Rebase

This change hides the computer identity in the LAN lobby. Originally it would expose the User login name and Computer name. Generally in pure Local Network matches this would be alright, but the Network Lobby is used for virtual private networks such as GameRanger and Radmin where we meet wild Internet users and generally want to keep our privacy.

The UI will still show computer information from retail game clients.

The identity was shown in this format:

loginname@COMPUTER_NAME

TODO

  • Add pull id to commits
  • Replicate in Generals

@xezon xezon added GUI For graphical user interface Major Severity: Minor < Major < Critical < Blocker Network Anything related to network, servers Gen Relates to Generals ZH Relates to Zero Hour labels Oct 13, 2025
Copy link

@Skyaero42 Skyaero42 left a comment

Choose a reason for hiding this comment

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

This is great!

@Mauller
Copy link

Mauller commented Oct 14, 2025

Does this also stop your machine sending your information?

@xezon
Copy link
Author

xezon commented Oct 14, 2025

Yes

@Stubbjax
Copy link

What is now shown instead?

@xezon
Copy link
Author

xezon commented Oct 15, 2025

When user does not tell its identity, then no tooltip is shown.
When user tells its identity (for example in RTS_DEBUG), then tooltip is shown.

@xezon xezon force-pushed the xezon/hide-computer-identity-lan branch from 54246df to 3122356 Compare October 15, 2025 17:09
@xezon
Copy link
Author

xezon commented Oct 15, 2025

Replicated in Generals without conflicts.

@xezon xezon merged commit 9ede47f into TheSuperHackers:main Oct 15, 2025
19 checks passed
@xezon xezon deleted the xezon/hide-computer-identity-lan branch October 15, 2025 17:28
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Nov 10, 2025
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Gen Relates to Generals GUI For graphical user interface Major Severity: Minor < Major < Critical < Blocker Network Anything related to network, servers ZH Relates to Zero Hour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants