Skip to content

chore: update game bridge to 1.45.9 #241

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

Merged
merged 2 commits into from
Jul 23, 2024
Merged

Conversation

platform-sa
Copy link
Collaborator

@platform-sa platform-sa commented Jul 22, 2024

Summary

The device code authentication works well on Android, iOS, and Mac, but the Unity SDK encounters issues on Windows. After logging in, the user's OIDC data stored in localStorage often disappears when reopening the game. This happens because Windows initially stores localStorage items in memory, unlike IndexedDB, which writes directly to disk. Upon looking at the Chromium source code for version 122.0.6261.130, local storage items only get committed after every 5 seconds. Additionally, this line is also the other cause of the issue because the metrics package was using local storage a lot.

To ensure better compatibility with game engines, IndexedDB should be used instead of local storage for storing OIDC data.

Update game bridge (build from ts-immutable-sdk version 1.45.9).

Customer Impact

Switched from localStorage to IndexedDB for storing OIDC data, resolving token persistence issues when reopening games and ensuring more reliable authentication across platforms.

@platform-sa platform-sa requested a review from a team July 22, 2024 02:52
@nattb8 nattb8 force-pushed the chore/update-game-bridge-1.45.9 branch from 5057132 to fccce30 Compare July 23, 2024 01:08
@nattb8 nattb8 merged commit 75fda87 into main Jul 23, 2024
15 checks passed
@nattb8 nattb8 deleted the chore/update-game-bridge-1.45.9 branch July 23, 2024 01:32
@github-actions github-actions bot added the chore label Jul 23, 2024
@nattb8 nattb8 added fix and removed chore labels Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants