Skip to content

Use Laravel for bearer token access#1894

Merged
williamjallen merged 1 commit intoKitware:masterfrom
williamjallen:laravel-bearer-token
Jan 10, 2024
Merged

Use Laravel for bearer token access#1894
williamjallen merged 1 commit intoKitware:masterfrom
williamjallen:laravel-bearer-token

Conversation

@williamjallen
Copy link
Collaborator

While looking through the documentation, I discovered that Laravel provides a Request::bearerToken() helper function.

@josephsnyder
Copy link
Member

@williamjallen, I am unable to submit with a bearer token even against the master branch in my development environment with both an admin and non-admin user. The test passes locally and in the CI, so I think we can merge. Any objections?

@williamjallen
Copy link
Collaborator Author

@williamjallen, I am unable to submit with a bearer token even against the master branch in my development environment with both an admin and non-admin user. The test passes locally and in the CI, so I think we can merge. Any objections?

Seems reasonable to me.

@williamjallen williamjallen added this pull request to the merge queue Jan 10, 2024
Merged via the queue into Kitware:master with commit 601c792 Jan 10, 2024
@williamjallen williamjallen deleted the laravel-bearer-token branch January 10, 2024 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments