Skip to content

Commit

Permalink
Merge pull request #23 from ronvanderheijden/bugfix-pipeline-issues
Browse files Browse the repository at this point in the history
Bugfix pipeline issues
  • Loading branch information
ronvanderheijden authored Apr 25, 2024
2 parents 9f56cce + 4a2d722 commit bd5f129
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 5 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ chmod 644 tmp/public.key
```

## Example
I recommand to [read this](https://oauth2.thephpleague.com/authorization-server/auth-code-grant/) first.
I recommend to [read this](https://oauth2.thephpleague.com/authorization-server/auth-code-grant/) first.

To enable OpenID Connect, follow these simple steps

Expand Down
2 changes: 2 additions & 0 deletions src/Laravel/DiscoveryController.php
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
<?php

declare(strict_types=1);

namespace OpenIDConnect\Laravel;

use Illuminate\Http\Request;
Expand Down
3 changes: 2 additions & 1 deletion src/Laravel/JwksController.php
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
<?php

declare(strict_types=1);

namespace OpenIDConnect\Laravel;

use Illuminate\Config\Repository as Config;
use Laravel\Passport\Passport;

class JwksController
Expand Down
2 changes: 1 addition & 1 deletion src/Laravel/config/openid.php
Original file line number Diff line number Diff line change
Expand Up @@ -59,5 +59,5 @@
* - /oauth/jwks
*/
'jwks' => true,
]
],
];
14 changes: 12 additions & 2 deletions src/Laravel/routes/web.php
Original file line number Diff line number Diff line change
@@ -1,7 +1,17 @@
<?php

declare(strict_types=1);

use Illuminate\Support\Facades\Route;
use OpenIDConnect\Laravel\DiscoveryController;
use OpenIDConnect\Laravel\JwksController;

if (config('openid.routes.discovery', true)) {
Route::get('/oauth/jwks', \OpenIDConnect\Laravel\JwksController::class)->name('openid.jwks');
Route::get('/oauth/jwks', JwksController::class)
->name('openid.jwks');
}

if (config('openid.routes.jwks', true)) {
Route::get('/.well-known/openid-configuration', \OpenIDConnect\Laravel\DiscoveryController::class)->name('openid.discovery');
Route::get('/.well-known/openid-configuration', DiscoveryController::class)
->name('openid.discovery');
}

0 comments on commit bd5f129

Please sign in to comment.