Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.x] Create getter for the middleware priority #37271

Merged
merged 2 commits into from
May 5, 2021

Conversation

roberto-aguilar
Copy link
Contributor

This getter will help to create tests to ensure that middlewares are registered in the right priority.

My use case for this is a middleware that I want to ensure gets registered after the StartSession but right before the AuthenticatesRequests one, so with this getter I can use their indexes to write a test for it.

This getter will help to create tests to ensure that middlewares are
registered in the right priority.

My use case for this is a middleware that I want to ensure gets
registered after the `StartSession` but right before the
`AuthenticatesRequests` one. With this getter I can use their indexes to
write a test for it.
@GrahamCampbell GrahamCampbell changed the title Create getter for the middleware priority [8.x] Create getter for the middleware priority May 5, 2021
@taylorotwell taylorotwell merged commit fe6e25e into laravel:8.x May 5, 2021
@roberto-aguilar roberto-aguilar deleted the middleware-priority branch May 5, 2021 23:08
ibrasho pushed a commit to ibrasho-forks/framework that referenced this pull request May 8, 2021
* Create getter for the middleware priority

This getter will help to create tests to ensure that middlewares are
registered in the right priority.

My use case for this is a middleware that I want to ensure gets
registered after the `StartSession` but right before the
`AuthenticatesRequests` one. With this getter I can use their indexes to
write a test for it.

* Update Kernel.php

Co-authored-by: Taylor Otwell <taylor@laravel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants