Routes

[mahr-martin]

Hello,
I'm using Nette 2.1.0 and route settings:

class RouterFactory {
    public function createRouter() {
        $router = new RouteList();

        // Register OPAuth routers
        $router[] = $opauthRouter = new RouteList('Opauth');
        \NetteOpauth\NetteOpauth::register($opauthRouter);

        $router[] = $adminRouter = new RouteList('Admin');
        $adminRouter[] = new Route('admin/<presenter>/<action>/[<id>]', 'Homepage:default');

        $router[] = $frontRouter = new RouteList('Public');
        $frontRouter[] = new Route('<presenter>/<action>[/<id>]', 'Homepage:default');

        $router[] = $frontRouter = new RouteList('Private');
        $frontRouter[] = new Route('private/<presenter>/<action>[/<id>]', 'Homepage:default');

        return $router;
    }
}

And it didn't work until I change \NetteOpauth\NetteOpauth::register() to:

public static function register($router) {
    $router[] = new Route('auth/logout', 'Auth:logout');
    $router[] = new Route('auth/callback', 'Auth:callback');
    $router[] = new Route('auth/<strategy>', 'Auth:auth');
    $router[] = new Route('auth/<strategy>/oauth2callback', 'Auth:auth');
    $router[] = new Route('auth/<strategy>/oauth_callback', 'Auth:auth');
    $router[] = new Route('auth/<strategy>/int_callback', 'Auth:auth');
}

• removed initial "/" from all routes. Is it a bug or is it my mistake?

[michalsvec]

What exactly didn't work? Works fine for me even on Nette 2.1.

[Marduke]

i use nette 2.2, only sandbox with added opauth
routing not works
solution is easy remove first '/' from start of route and everything works, maybe change in 2.2 as wrote before

$router[] = new Route('/auth/logout', 'Auth:logout');....
====>
$router[] = new Route('auth/logout', 'Auth:logout');....

[michalsvec]

I will check and let you know.

[michalsvec]

You are right. Trailing slashes are useless.
Fixed in https://github.com/michalsvec/nette-opauth/releases/tag/v0.6.6