Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds support for external file-based configuration of the LootLocker SDK, allowing settings to be loaded from a .bytes file at initialization. The implementation includes optional encryption for the configuration file and makes the SDK package name configurable.
Changes:
- Introduced
ExternalFileConfigclass for JSON-serializable SDK configuration - Added encryption utilities for optional config file obfuscation
- Refactored
LootLockerConfigto check for and load external file configurations on initialization - Updated log messages throughout the SDK to remove hardcoded "LootLocker" references, making them more generic
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| Runtime/Game/Utilities/LootLockerEncryptionUtilities.cs | Implements AES encryption/decryption utilities for config files |
| Runtime/Game/Utilities/LootLockerEncryptionUtilities.cs.meta | Unity metadata for encryption utilities |
| Runtime/Game/Resources/LootLockerExternalFileConfig.cs | Defines serializable external configuration structure |
| Runtime/Game/Resources/LootLockerExternalFileConfig.cs.meta | Unity metadata for external config |
| Runtime/Game/Resources/LootLockerConfig.cs | Refactored to support file-based configuration loading with encryption support |
| Runtime/Game/Requests/WhiteLabelRequest.cs | Updated error messages to remove "LootLocker" prefix |
| Runtime/Game/LootLockerSDKManager.cs | Updated log messages for generic SDK references |
| Runtime/Game/LootLockerLogger.cs | Added configurable log label functionality |
| Runtime/Editor/LogViewer/LootLockerLogViewerUI.cs | Updated to handle new log label format |
| Runtime/Editor/Editor UI/LootLockerAdminExtension.cs | Fixed formatting spacing |
| Runtime/Client/LootLockerLifecycleManager.cs | Updated initialization log message |
| Runtime/Client/LootLockerHTTPClient.cs | Updated HTTP client log messages |
| Runtime/Client/LootLockerEventSystem.cs | Updated event system log message |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
JohannesLoot
approved these changes
Feb 18, 2026
Contributor
JohannesLoot
left a comment
There was a problem hiding this comment.
Only remark is the comment that I made, if that is intentional and I have missed some context, then merge away!
9508b5e to
fc57006
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
File Configurable SDK
This is the first step of the publisher package. What it does is it allows for a
.bytesfile to be automatically read by the SDK on boot. If it exists, then that file is used to set the LootLocker SDK configuration.Solves issue: https://github.com/lootlocker/index/issues/1260
Changes
LootLockerExternalFileConfigas a json serializable version of LootLockerConfigExtra Features