Jingga/phpOMS
1
0
Fork 0
mirror of https://github.com/Karaka-Management/phpOMS.git synced 2026-02-10 22:18:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Reduce match overhead

Browse Source
This commit is contained in:
Dennis Eichhorn 2019-04-12 22:55:09 +02:00
parent 35360ef7a8
commit 95869f70c4
1 changed files with 8 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
Router/Router.php
View File

@ -100,16 +100,21 @@ final class Router
* *
* @return array[] * @return array[]
* *
* @throws \InvalidArgumentException
*
* @since 1.0.0 * @since 1.0.0
*/ */
public function route(string $request, int $verb = RouteVerb::GET, string $app = null, int $orgId = null, $account = null) : array public function route(string $request, int $verb = RouteVerb::GET, string $app = null, int $orgId = null, $account = null) : array
{ {
$bound = []; $bound = [];
foreach ($this->routes as $route => $destination) { foreach ($this->routes as $route => $destination) {
if (!((bool) \preg_match('~^' . $route . '$~', $request))) {
continue;
}
foreach ($destination as $d) { foreach ($destination as $d) {
if ($this->match($route, $d['verb'], $request, $verb)) { if ($d['verb'] === RouteVerb::ANY
|| $verb === RouteVerb::ANY
|| ($verb & $d['verb']) === $verb
) {
// if csrf is required but not set // if csrf is required but not set
if (isset($d['csrf']) && !$d['csrf']) { if (isset($d['csrf']) && !$d['csrf']) {
\array_merge($bound, $this->route('/' . $app . '/e403', $verb)); \array_merge($bound, $this->route('/' . $app . '/e403', $verb));
@ -134,21 +139,4 @@ final class Router
return $bound; return $bound;
} }
/**
* Match route and uri.
*
* @param string $route Route
* @param int $routeVerb GET,POST for this route
* @param string $uri Uri
* @param int $remoteVerb Verb this request is using
*
* @return bool
*
* @since 1.0.0
*/
private function match(string $route, int $routeVerb, string $uri, int $remoteVerb = RouteVerb::GET) : bool
{
return (bool) \preg_match('~^' . $route . '$~', $uri) && ($routeVerb === RouteVerb::ANY || $remoteVerb === RouteVerb::ANY || ($remoteVerb & $routeVerb) === $remoteVerb);
}
} }