Skip to content

Conversation

@lorux0
Copy link
Collaborator

@lorux0 lorux0 commented Oct 1, 2025

What does this PR change?

Integrates the auto-login architecture. Shape
The implementation includes:

  • Token file parsing located at /{userAppData}/DecentralandLauncherLight/auth-token-bridge.txt (depending on the platform).
  • Authentication server requests that gets the identity json.
  • Integration with the startup flow.

Additionally, due to cyclic dependencies, all of the authenticator implementations have been moved into a new asmref.

Quality Checklist

  • Changes have been tested locally
  • Documentation has been updated (if required)
  • Performance impact has been considered
  • For SDK features: Test scene is included

Code Review Reference

Please review our Code Review Standards before submitting.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 1, 2025

@lorux0 lorux0 marked this pull request as ready for review October 14, 2025 12:11
@lorux0 lorux0 requested review from a team as code owners October 14, 2025 12:11
@balducciv
Copy link

balducciv commented Oct 15, 2025

Testing in progress

Test on Windows

Auto-login

test.windows.PR-5636.windows.mp4

Player (1).log
Player-prev (1).log

After auto-login, log out, login with a different user and the auto-login corresponds to the user last logged in

test.windows.PR-5636.windows.-.switch.user.mp4

Player.log
Player-prev.log

Switch user - choose "Use a different account"

test.windows.PR-5636.windows.-.use.a.different.account.mp4

Player.log

Player-prev.log

**Test in Mac **

Auto login

auto.login.mac.mp4

Switch to another account and auto-login

auto.login.mac.-.another.account.mp4

Player.log

Player-prev.log

Smoke test has also been performed on both platforms on the following areas:

[ ✔️ ] Log In/Log Out
[ ✔️ ] Backpack and wearables in world
[ ✔️ ] Emotes in world and in backpack
[ ✔️ ] Teleport with map/coordinates/Jump In
[ ✔️ ] Chat and multiplayer
[ ✔️ ] Profile card
[ ✔️ ] Camera
[ ✔️ ] Skybox
[ ✔️ ] Settings

Copy link
Contributor

@NickKhalow NickKhalow left a comment

Choose a reason for hiding this comment

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

LGTM

@lorux0 lorux0 enabled auto-merge (squash) October 16, 2025 14:15
@lorux0 lorux0 merged commit 2667928 into dev Oct 16, 2025
12 of 15 checks passed
@lorux0 lorux0 deleted the feat/auto-login branch October 16, 2025 15:37
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.

5 participants