MC-API / Issues

Public issue tracker for mc-api.net
https://mc-api.net
2 stars 0 forks source link

The cache system in UUID to Name store the query instead of the actual name #34

Closed KareemMAX closed 8 years ago

KareemMAX commented 8 years ago

When I tried to use the api the first time looked like this

{"full_uuid":"c054d78e-e15a-4633-be16-6650c0855efb","source":"mojang","query":"kareemmax","name":"KareemMAX","took":254,"uuid":"c054d78ee15a4633be166650c0855efb"}

the second time looked like this

{"full_uuid":"c054d78e-e15a-4633-be16-6650c0855efb","source":"cache","query":"kareemmax","name":"kareemmax","took":1,"uuid":"c054d78ee15a4633be166650c0855efb"}

Please solve this as soon as you can @ImNatho

nathantowell commented 8 years ago

Wait. What part is the issue? One was loaded from our cache and the other was retrieved from mojang's servers? Or is the the "name" value?

KareemMAX commented 8 years ago

Your server seems to be save the query in the cache in the "name" value so when I try to get it from the cache the "name" value is the previous query I tried to use different name and different times and in lowercase or uppercase and the problem still the same

nathantowell commented 8 years ago

Okay, that's a quick fix. I'll add it in before the skins section

KareemMAX commented 8 years ago

The mojang one was correct, the problem with the cache system

KareemMAX commented 8 years ago

Ok I'm waiting

nathantowell commented 8 years ago

@KareemMAX I have UNI today so It might take a few hours before this is merged into master.

KareemMAX commented 8 years ago

No problem I will change my code when you done

nathantowell commented 8 years ago

Pushed to master. Waiting for tests and deploy.

nathantowell commented 8 years ago

Deployed