Jérémy Munsch
81aabdab72
This removes api authentification with token in favor of signature method. Refactorize code and clean up. It add more unit test plus code coverage in test/converage. - Added config.api.timestampTimeout - Added Helper::signRequest This calculates the request signature - Added more unit tests and extend code coverage. - Added rate limiter for web routes - Added RemoveSpaces Sanitizer on api requests - Added stress test suite - Changed AuthServiceProvider api-token to api-hmac - Changed CustomerSeeder to have at least of customer activated (for unit tests) - Changed used transation for saveing multiple models - Changed, in ServiceController generateApiToken to generateServicePassword - Changed, timestamp has to be send in request with a DateTime format, it also has to a UTC value - Composer added brianium/paratest - Composer update - Fixed case where removing single optin could be the wrong one (not for the good service) - Fixed customer delete - Fixed customer would not be resolved by brand. - Fixed Really check that request is json checking body content, preventing sending malformed requests - Remove ApiError::needAuth - Removed api_token - Removed ApiController::authenticate - Removed config.api.tokenTimeout - Removed Header check or usage because of HAProxy doing weird stuff - Removed lock files from gitignore - Removed memcached dependency - Removed unused questions resources api routes - Removed workspace file Migrations - Added unique index on services.bearer_token - Changed customer_id on customer_answer is nullable - Changed services.bearer_pwd length to 312 (the size of the token) - Dropped services.api_token and service.api_token_date
84 lines
2.2 KiB
PHP
84 lines
2.2 KiB
PHP
<?php
|
|
|
|
namespace Tests\Client;
|
|
|
|
use App\Lib\Helper;
|
|
use Illuminate\Foundation\Testing\TestCase;
|
|
use Illuminate\Http\Request;
|
|
use Illuminate\Support\Facades\Route;
|
|
use Tests\CreatesApplication;
|
|
|
|
class HelperTest extends TestCase
|
|
{
|
|
use CreatesApplication;
|
|
|
|
/**
|
|
* Helper::getRoute test
|
|
*
|
|
* @return void
|
|
*/
|
|
public function testGetRoute()
|
|
{
|
|
$request = $this->makeRequest('apiV2.ping');
|
|
$this->assertInstanceOf(\Illuminate\Routing\Route::class, Helper::getRoute($request));
|
|
}
|
|
|
|
/**
|
|
* Helper::getRoutePrefix test
|
|
*
|
|
* @return void
|
|
*/
|
|
public function testGetRoutePrefix()
|
|
{
|
|
$request = $this->makeRequest('apiV2.ping');
|
|
$this->assertEquals($request->route()->getPrefix(), Helper::getRoutePrefix($request));
|
|
}
|
|
|
|
/**
|
|
* Helper::getRouteName test
|
|
*
|
|
* @return void
|
|
*/
|
|
public function testGetRouteName()
|
|
{
|
|
$request = $this->makeRequest('apiV2.ping');
|
|
$this->assertEquals($request->route()->getName(), Helper::getRouteName($request));
|
|
}
|
|
|
|
/**
|
|
* Helper::isRequestForApiV1 test
|
|
*
|
|
* @return void
|
|
*/
|
|
public function testIsRequestForApiV1()
|
|
{
|
|
$request = $this->makeRequest('App\Http\Controllers\Api\V1\CustomerController@activateAccount');
|
|
$this->assertTrue(Helper::isRequestForApiV1($request));
|
|
}
|
|
|
|
/**
|
|
* Helper::isRequestForApiV2 test
|
|
*
|
|
* @return void
|
|
*/
|
|
public function testIsRequestForApiV2()
|
|
{
|
|
$request = $this->makeRequest('apiV2.ping');
|
|
$this->assertTrue(Helper::isRequestForApiV2($request));
|
|
}
|
|
|
|
/**
|
|
* Make laravel request using name or action.
|
|
*
|
|
* @param string $nameOrAction
|
|
* @return \Illuminate\Http\Request
|
|
*/
|
|
private function makeRequest(string $nameOrAction): Request
|
|
{
|
|
$route = Route::getRoutes()->getByName($nameOrAction) ?? Route::getRoutes()->getByAction($nameOrAction);
|
|
$request = new Request([], [], [], [], [], ['REQUEST_URI' => $route->uri()]);
|
|
return $request->setRouteResolver(function () use ($route, $request) {
|
|
return ($route)->bind($request);
|
|
});
|
|
}
|
|
}
|