Add a redirect for new Usernames to UUIDs endpoint (#29)

* fix: add redirect for new "Usernames to UUIDs" endpoint location

"As of 23w42a, the endpoint moved to: 'POST https://sessionserver.mojang.com/session/minecraft/profile/lookup/bulk/byname'
"
https://wiki.vg/Mojang_API#Usernames_to_UUIDs

* add a redirect for authlib-injector too

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>

* revert: .gitignore

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>

* tests: pass new URL to existing tests

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>

* review: address requested changes

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>

* style: add type hints to MinecraftProfilesCest functions too

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>

---------

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>
This commit is contained in:
Octol1ttle 2023-11-20 05:36:39 +05:00 committed by GitHub
parent 667f582b3e
commit 574d03d02f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 103 additions and 48 deletions

View File

@ -41,6 +41,7 @@ return [
'/minecraft/session/hasJoined' => 'session/session/has-joined', '/minecraft/session/hasJoined' => 'session/session/has-joined',
'/minecraft/session/legacy/hasJoined' => 'session/session/has-joined-legacy', '/minecraft/session/legacy/hasJoined' => 'session/session/has-joined-legacy',
'/minecraft/session/profile/<uuid>' => 'session/session/profile', '/minecraft/session/profile/<uuid>' => 'session/session/profile',
'POST /minecraft/session/profile/lookup/bulk/byname' => 'mojang/api/uuids-by-usernames',
// Mojang API module routes // Mojang API module routes
'/mojang/profiles/<username>' => 'mojang/api/uuid-by-username', '/mojang/profiles/<username>' => 'mojang/api/uuid-by-username',
@ -54,4 +55,5 @@ return [
'/authlib-injector/sessionserver/session/minecraft/hasJoined' => 'session/session/has-joined', '/authlib-injector/sessionserver/session/minecraft/hasJoined' => 'session/session/has-joined',
'/authlib-injector/sessionserver/session/minecraft/profile/<uuid>' => 'session/session/profile', '/authlib-injector/sessionserver/session/minecraft/profile/<uuid>' => 'session/session/profile',
'/authlib-injector/api/profiles/minecraft' => 'mojang/api/uuids-by-usernames', '/authlib-injector/api/profiles/minecraft' => 'mojang/api/uuids-by-usernames',
'/authlib-injector/sessionserver/session/minecraft/profile/lookup/bulk/byname' => 'mojang/api/uuids-by-usernames',
]; ];

View File

@ -11,9 +11,4 @@ class MojangApiRoute extends BasePage {
public function usernamesByUuid($uuid) { public function usernamesByUuid($uuid) {
$this->getActor()->sendGET("/api/mojang/profiles/{$uuid}/names"); $this->getActor()->sendGET("/api/mojang/profiles/{$uuid}/names");
} }
public function uuidsByUsernames($uuids) {
$this->getActor()->sendPOST('/api/mojang/profiles', $uuids);
}
} }

View File

@ -4,11 +4,15 @@ declare(strict_types=1);
namespace api\tests\functional\authlibInjector; namespace api\tests\functional\authlibInjector;
use api\tests\FunctionalTester; use api\tests\FunctionalTester;
use Codeception\Example;
final class MinecraftProfilesCest { final class MinecraftProfilesCest {
public function geUuidByOneUsername(FunctionalTester $I) { /**
$I->sendPOST('/api/authlib-injector/api/profiles/minecraft', ['Admin']); * @dataProvider bulkProfilesEndpoints
*/
public function getUuidByOneUsername(FunctionalTester $I, Example $url) : void {
$I->sendPOST($url[0], ['Admin']);
$I->canSeeResponseCodeIs(200); $I->canSeeResponseCodeIs(200);
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
[ [
@ -18,21 +22,32 @@ final class MinecraftProfilesCest {
]); ]);
} }
public function getUuidsByUsernames(FunctionalTester $I) { /**
$I->sendPOST('/api/authlib-injector/api/profiles/minecraft', ['Admin', 'AccWithOldPassword', 'Notch']); * @dataProvider bulkProfilesEndpoints
*/
public function getUuidsByUsernames(FunctionalTester $I, Example $url) : void {
$I->sendPOST($url[0], ['Admin', 'AccWithOldPassword', 'Notch']);
$this->validateFewValidUsernames($I); $this->validateFewValidUsernames($I);
} }
public function getUuidsByUsernamesWithPostString(FunctionalTester $I) {
/**
* @dataProvider bulkProfilesEndpoints
*/
public function getUuidsByUsernamesWithPostString(FunctionalTester $I, Example $url) : void {
$I->sendPOST( $I->sendPOST(
'/api/authlib-injector/api/profiles/minecraft', $url[0],
json_encode(['Admin', 'AccWithOldPassword', 'Notch']), json_encode(['Admin', 'AccWithOldPassword', 'Notch']),
); );
$this->validateFewValidUsernames($I); $this->validateFewValidUsernames($I);
} }
public function getUuidsByPartialNonexistentUsernames(FunctionalTester $I) {
$I->sendPOST('/api/authlib-injector/api/profiles/minecraft', ['Admin', 'DeletedAccount', 'not-exists-user']); /**
* @dataProvider bulkProfilesEndpoints
*/
public function getUuidsByPartialNonexistentUsernames(FunctionalTester $I, Example $url) : void {
$I->sendPOST($url[0], ['Admin', 'DeletedAccount', 'not-exists-user']);
$I->canSeeResponseCodeIs(200); $I->canSeeResponseCodeIs(200);
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
[ [
@ -44,20 +59,28 @@ final class MinecraftProfilesCest {
$I->cantSeeResponseJsonMatchesJsonPath('$.[?(@.name="not-exists-user")]'); $I->cantSeeResponseJsonMatchesJsonPath('$.[?(@.name="not-exists-user")]');
} }
public function passAllNonexistentUsernames(FunctionalTester $I) {
$I->sendPOST('/api/authlib-injector/api/profiles/minecraft', ['not-exists-1', 'not-exists-2']); /**
* @dataProvider bulkProfilesEndpoints
*/
public function passAllNonexistentUsernames(FunctionalTester $I, Example $url) : void {
$I->sendPOST($url[0], ['not-exists-1', 'not-exists-2']);
$I->canSeeResponseCodeIs(200); $I->canSeeResponseCodeIs(200);
$I->canSeeResponseIsJson(); $I->canSeeResponseIsJson();
$I->canSeeResponseEquals('[]'); $I->canSeeResponseEquals('[]');
} }
public function passTooManyUsernames(FunctionalTester $I) {
/**
* @dataProvider bulkProfilesEndpoints
*/
public function passTooManyUsernames(FunctionalTester $I, Example $url) : void {
$usernames = []; $usernames = [];
for ($i = 0; $i < 150; $i++) { for ($i = 0; $i < 150; $i++) {
$usernames[] = random_bytes(10); $usernames[] = random_bytes(10);
} }
$I->sendPOST('/api/authlib-injector/api/profiles/minecraft', $usernames); $I->sendPOST($url[0], $usernames);
$I->canSeeResponseCodeIs(400); $I->canSeeResponseCodeIs(400);
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
'error' => 'IllegalArgumentException', 'error' => 'IllegalArgumentException',
@ -65,8 +88,12 @@ final class MinecraftProfilesCest {
]); ]);
} }
public function passEmptyUsername(FunctionalTester $I) {
$I->sendPOST('/api/authlib-injector/api/profiles/minecraft', ['Admin', '']); /**
* @dataProvider bulkProfilesEndpoints
*/
public function passEmptyUsername(FunctionalTester $I, Example $url) : void {
$I->sendPOST($url[0], ['Admin', '']);
$I->canSeeResponseCodeIs(400); $I->canSeeResponseCodeIs(400);
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
'error' => 'IllegalArgumentException', 'error' => 'IllegalArgumentException',
@ -74,8 +101,12 @@ final class MinecraftProfilesCest {
]); ]);
} }
public function passEmptyField(FunctionalTester $I) {
$I->sendPOST('/api/authlib-injector/api/profiles/minecraft', []); /**
* @dataProvider bulkProfilesEndpoints
*/
public function passEmptyField(FunctionalTester $I, Example $url) : void {
$I->sendPOST($url[0], []);
$I->canSeeResponseCodeIs(400); $I->canSeeResponseCodeIs(400);
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
'error' => 'IllegalArgumentException', 'error' => 'IllegalArgumentException',
@ -83,7 +114,7 @@ final class MinecraftProfilesCest {
]); ]);
} }
private function validateFewValidUsernames(FunctionalTester $I) { private function validateFewValidUsernames(FunctionalTester $I) : void {
$I->canSeeResponseCodeIs(200); $I->canSeeResponseCodeIs(200);
$I->canSeeResponseIsJson(); $I->canSeeResponseIsJson();
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
@ -102,4 +133,11 @@ final class MinecraftProfilesCest {
]); ]);
} }
private function bulkProfilesEndpoints() : array {
return [
["/api/authlib-injector/api/profiles/minecraft"],
["/api/authlib-injector/sessionserver/session/minecraft/profile/lookup/bulk/byname"]
];
}
} }

View File

@ -1,23 +1,16 @@
<?php <?php
namespace api\tests\functional\authserver; namespace api\tests\functional\authserver;
use api\tests\_pages\MojangApiRoute;
use api\tests\FunctionalTester; use api\tests\FunctionalTester;
use Codeception\Example;
class UsernamesToUuidsCest { class UsernamesToUuidsCest {
/** /**
* @var MojangApiRoute * @dataProvider bulkProfilesEndpoints
*/ */
private $route; public function getUuidByOneUsername(FunctionalTester $I, Example $url) : void {
public function _before(FunctionalTester $I) {
$this->route = new MojangApiRoute($I);
}
public function geUuidByOneUsername(FunctionalTester $I) {
$I->wantTo('get uuid by one username'); $I->wantTo('get uuid by one username');
$this->route->uuidsByUsernames(['Admin']); $I->sendPost($url[0], ['Admin']);
$I->canSeeResponseCodeIs(200); $I->canSeeResponseCodeIs(200);
$I->canSeeResponseIsJson(); $I->canSeeResponseIsJson();
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
@ -28,21 +21,30 @@ class UsernamesToUuidsCest {
]); ]);
} }
public function getUuidsByUsernames(FunctionalTester $I) { /**
* @dataProvider bulkProfilesEndpoints
*/
public function getUuidsByUsernames(FunctionalTester $I, Example $url) : void {
$I->wantTo('get uuids by few usernames'); $I->wantTo('get uuids by few usernames');
$this->route->uuidsByUsernames(['Admin', 'AccWithOldPassword', 'Notch']); $I->sendPost($url[0], ['Admin', 'AccWithOldPassword', 'Notch']);
$this->validateFewValidUsernames($I); $this->validateFewValidUsernames($I);
} }
public function getUuidsByUsernamesWithPostString(FunctionalTester $I) { /**
* @dataProvider bulkProfilesEndpoints
*/
public function getUuidsByUsernamesWithPostString(FunctionalTester $I, Example $url) : void {
$I->wantTo('get uuids by few usernames'); $I->wantTo('get uuids by few usernames');
$this->route->uuidsByUsernames(json_encode(['Admin', 'AccWithOldPassword', 'Notch'])); $I->sendPost($url[0], json_encode(['Admin', 'AccWithOldPassword', 'Notch']));
$this->validateFewValidUsernames($I); $this->validateFewValidUsernames($I);
} }
public function getUuidsByPartialNonexistentUsernames(FunctionalTester $I) { /**
* @dataProvider bulkProfilesEndpoints
*/
public function getUuidsByPartialNonexistentUsernames(FunctionalTester $I, Example $url) : void {
$I->wantTo('get uuids by few usernames and some nonexistent'); $I->wantTo('get uuids by few usernames and some nonexistent');
$this->route->uuidsByUsernames(['Admin', 'DeletedAccount', 'not-exists-user']); $I->sendPost($url[0], ['Admin', 'DeletedAccount', 'not-exists-user']);
$I->canSeeResponseCodeIs(200); $I->canSeeResponseCodeIs(200);
$I->canSeeResponseIsJson(); $I->canSeeResponseIsJson();
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
@ -55,22 +57,28 @@ class UsernamesToUuidsCest {
$I->cantSeeResponseJsonMatchesJsonPath('$.[?(@.name="not-exists-user")]'); $I->cantSeeResponseJsonMatchesJsonPath('$.[?(@.name="not-exists-user")]');
} }
public function passAllNonexistentUsernames(FunctionalTester $I) { /**
* @dataProvider bulkProfilesEndpoints
*/
public function passAllNonexistentUsernames(FunctionalTester $I, Example $url) : void {
$I->wantTo('get specific response when pass all nonexistent usernames'); $I->wantTo('get specific response when pass all nonexistent usernames');
$this->route->uuidsByUsernames(['not-exists-1', 'not-exists-2']); $I->sendPost($url[0], ['not-exists-1', 'not-exists-2']);
$I->canSeeResponseCodeIs(200); $I->canSeeResponseCodeIs(200);
$I->canSeeResponseIsJson(); $I->canSeeResponseIsJson();
$I->canSeeResponseContainsJson([]); $I->canSeeResponseContainsJson([]);
} }
public function passTooManyUsernames(FunctionalTester $I) { /**
* @dataProvider bulkProfilesEndpoints
*/
public function passTooManyUsernames(FunctionalTester $I, Example $url) : void {
$I->wantTo('get specific response when pass too many usernames'); $I->wantTo('get specific response when pass too many usernames');
$usernames = []; $usernames = [];
for ($i = 0; $i < 150; $i++) { for ($i = 0; $i < 150; $i++) {
$usernames[] = random_bytes(10); $usernames[] = random_bytes(10);
} }
$this->route->uuidsByUsernames($usernames); $I->sendPost($url[0], $usernames);
$I->canSeeResponseCodeIs(400); $I->canSeeResponseCodeIs(400);
$I->canSeeResponseIsJson(); $I->canSeeResponseIsJson();
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
@ -79,9 +87,12 @@ class UsernamesToUuidsCest {
]); ]);
} }
public function passEmptyUsername(FunctionalTester $I) { /**
* @dataProvider bulkProfilesEndpoints
*/
public function passEmptyUsername(FunctionalTester $I, Example $url) : void {
$I->wantTo('get specific response when pass empty username'); $I->wantTo('get specific response when pass empty username');
$this->route->uuidsByUsernames(['Admin', '']); $I->sendPost($url[0], ['Admin', '']);
$I->canSeeResponseCodeIs(400); $I->canSeeResponseCodeIs(400);
$I->canSeeResponseIsJson(); $I->canSeeResponseIsJson();
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
@ -90,9 +101,12 @@ class UsernamesToUuidsCest {
]); ]);
} }
public function passEmptyField(FunctionalTester $I) { /**
* @dataProvider bulkProfilesEndpoints
*/
public function passEmptyField(FunctionalTester $I, Example $url) : void {
$I->wantTo('get response when pass empty array'); $I->wantTo('get response when pass empty array');
$this->route->uuidsByUsernames([]); $I->sendPost($url[0], []);
$I->canSeeResponseCodeIs(400); $I->canSeeResponseCodeIs(400);
$I->canSeeResponseIsJson(); $I->canSeeResponseIsJson();
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
@ -101,7 +115,7 @@ class UsernamesToUuidsCest {
]); ]);
} }
private function validateFewValidUsernames(FunctionalTester $I) { private function validateFewValidUsernames(FunctionalTester $I) : void {
$I->canSeeResponseCodeIs(200); $I->canSeeResponseCodeIs(200);
$I->canSeeResponseIsJson(); $I->canSeeResponseIsJson();
$I->canSeeResponseContainsJson([ $I->canSeeResponseContainsJson([
@ -120,4 +134,10 @@ class UsernamesToUuidsCest {
]); ]);
} }
private function bulkProfilesEndpoints() : array {
return [
["/api/authlib-injector/api/profiles/minecraft"],
["/api/authlib-injector/sessionserver/session/minecraft/profile/lookup/bulk/byname"]
];
}
} }