Skip to content
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

Map of Xaero's World Map gets reset when joining secure servers #156

Closed
RedstoneDubstep opened this issue Aug 12, 2022 · 0 comments
Closed
Assignees
Labels
3rd party issue The issue is not related to this mod alone bug Something isn't working confirmed Confirmed bug/approved suggestion low priority

Comments

@RedstoneDubstep
Copy link

Modloader

Forge

Minecraft Version

1.19.1

Modloader Version

Forge 43.0.8

No Chat Reports Version

1.19.2-v1.4.0

Modpack Info

No response

The latest.log File

https://pastebin.com/mWLAz0V2

Bug Description

When joining a server with enforce-secure-profile=true with Xaero's World Map and NCR, the previously uncovered world map will be reset. This gets undone when re-joining the server without NCR, in this instance the previous world map is accessible again. Looking into this, this is likely caused by Xaero's World Map not being able to query the IP adress of the server and thus not knowing in what folder the world map for that server is stored. The IP (and, in fact, all other data the client is supposed to know about the current server) doesn't get set properly when connecting to a server with enforce-secure-profile=true with NCR due to it not passing the ServerData argument to ConnectScreen#startConnecting (see NoChatReportsClient#reconnectLastServer).

Steps to Reproduce

  1. Start a Forge client with Xaero's World Map installed, and a local Forge server without any mods and the enforce-secure-profile property set to true.
  2. Join the server and fly around so chunks get added to the world map, then close the client.
  3. Start the client again, this time with Xaero's World Map and this mod installed, and join the server (by clicking the "Proceed" button in NCR's warning screen).
  4. Observe that the previous world map is not accessible anymore, and instead a new, blank world map is started.
  5. Close the client, remove NCR, re-join the server, and observe the world map being accessible again.

Other Information

Tested this with Xaero's World Map 1.26.2.
If you want to fix this, I'd propose either trying to access the currentServer field of Minecraft to parse it into the mentioned method (though I don't know whether that field is already reset when the DisconnectionScreen is opened) or storing the server data alongside the server adress in ServerSafetyState.

@RedstoneDubstep RedstoneDubstep added the bug Something isn't working label Aug 12, 2022
@Aizistral Aizistral added 3rd party issue The issue is not related to this mod alone forge This issue is only relevant for Forge version of the mod low priority confirmed Confirmed bug/approved suggestion and removed forge This issue is only relevant for Forge version of the mod labels Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3rd party issue The issue is not related to this mod alone bug Something isn't working confirmed Confirmed bug/approved suggestion low priority
Projects
None yet
Development

No branches or pull requests

2 participants