Infinite loop after upgrade

after upgrading from 6 to 7 when navigating to admin/login (the nova login page) i get an infinite loop of [302]: GET /admin/login, does anyone know what may be causing this?

    public function withAuthenticationRoutes($middleware = ['web'])
    {
        Route::namespace('Laravel\Nova\Http\Controllers')
            ->domain(config('nova.domain', null))
            ->middleware($middleware)
            ->as('nova.')
            ->prefix(Nova::path())
            ->group(function () {
//                Route::get('/login', 'LoginController@showLoginForm');
                Route::post('/login', 'LoginController@login')->name('login');
            });

        Route::namespace('Laravel\Nova\Http\Controllers')
            ->domain(config('nova.domain', null))
            ->middleware(config('nova.middleware', []))
            ->as('nova.')
            ->prefix(Nova::path())
            ->group(function () {
                Route::get('/logout', 'LoginController@logout')->name('logout');
            });

        return $this;
    }

its no longer hitting the line i have commented out in PendingRouteRegistration and is now hitting the one in the vendor files.