limosa-io / laravel-openid-connect-server

This is an OpenID Connect Server for Laravel.
GNU Affero General Public License v3.0
53 stars 22 forks source link

Installation failes because of missing columns #22

Open michaelklopf opened 1 year ago

michaelklopf commented 1 year ago

I followed the installation steps

composer require nl.idaas/laravel-openid-connect
php artisan migrate
php artisan passport:install --uuids
php artisan vendor:publish --provider="Idaas\Passport\PassportServiceProvider" --force

And when running

php artisan passport:install --uuids

it throws the errors

Column not found: 1054 Unknown column 'provider' in 'field list' (Connection: mysql, SQL: insert into `oidc_clients` (`user_id`, `name`, `secret`, `provider`, `redirect`, `personal_access_client`, `password_client`, `revoked`, `client_id`, `updated_at`, `created_at`) values (?, Laravel Personal Access Client, ngXZEOEtQT6wbu32edq4ZqlUZszDEezn7MkQjIh9, ?, http://localhost, 1, 0, 0, 98bf710f-e227-435e-97e0-17e82fcaceac, 2023-03-22 10:13:50, 2023-03-22 10:13:50))

and

Column not found: 1054 Unknown column 'redirect' in 'field list' (Connection: mysql, SQL: insert into `oidc_clients` (`user_id`, `name`, `secret`, `provider`, `redirect`, `personal_access_client`, `password_client`, `revoked`, `client_id`, `updated_at`, `created_at`) values (?, Laravel Personal Access Client, 793Mh1Yt1J0irPDAr9m6pffPx0Y9S1qnqH0Hj2ED, ?, http://localhost, 1, 0, 0, 98bf717c-7be6-40f9-952a-a78ff26f1cc1, 2023-03-22 10:15:01, 2023-03-22 10:15:01))

I guess that passport has made updates that are not reflected in this package yet?

The problem is solved when adding

            $table->string('provider')->nullable();

            $table->text('redirect');

to the CreateOidcClientsTable migration.

yuqihc commented 2 weeks ago

I encountered the same problem "nl.idaas/laravel-openid-connect": "^0.2.2"