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

[BUG] Summoner-V4 returns extra fields (from LCU) #1007

Open
renja-g opened this issue Nov 8, 2024 · 1 comment
Open

[BUG] Summoner-V4 returns extra fields (from LCU) #1007

renja-g opened this issue Nov 8, 2024 · 1 comment

Comments

@renja-g
Copy link

renja-g commented Nov 8, 2024

I request and merge Account-V1 with Summoner-V4 every 10 sec, for about 10 summoners.

This has been running for about 2 weeks without any problems, but now suddenly this shows up for 2 summoners in the DB:

{
  "_id": {
    "$oid": "6726a6b20bcca0fd0c053ad7"
  },
  "gameName": "WeingottBachus",
  "tagLine": "EUW",
  "platform": "euw1",
  "accountId": 239745070,
+ "puuid": "c7ea728e-8999-5e32-abdc-8d44b8417642",
  "profileIconId": 5290,
  "revisionDate": {
    "$numberLong": "1730922591866"
  },
  "summonerLevel": 672,
  "summonerId": 107818328,
+ "expPoints": 503,
+ "expToNextLevel": 3552,
  "internalName": "weingottbachus",
+ "lastGameDate": {
    "$numberLong": "1730922591866"
  },
+ "level": 672,
+ "levelAndXpVersion": 8068,
  "name": "WeingottBachus",
+ "nameChangeFlag": false,
+ "privacy": "PUBLIC",
+ "revisionId": 0,
+ "unnamed": false
}

Everything not marked with a + is either added from me or MongoDB

With the puuid change to an UUID-V5 my code ofc. does not run anymore...
I think I had this problem about 2 weeks ago, that a puuid of a different summoner changed, but I didn’t pay attention to other fields, since I was sill in active development I just thought I messed up something.

If I now request the summoner's data, I get the normal response.

The full code can be looked at Realm-Warp
The important part is here:
https://github.com/renja-g/Realm-Warp/blob/8bf9bc38153045f8cf9d9b665c7095aa60aadc08/watcher/main.py#L124-L141

@renja-g
Copy link
Author

renja-g commented Nov 13, 2024

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

No branches or pull requests

1 participant