Skip to content
This repository was archived by the owner on Feb 7, 2024. It is now read-only.

Add the ability to configure middleware. #28

Closed
wants to merge 2 commits into from
Closed

Add the ability to configure middleware. #28

wants to merge 2 commits into from

Conversation

irazasyed
Copy link
Contributor

Fixes #22

@irazasyed
Copy link
Contributor Author

@mpociot @freekmurze Can we have some attention on this issue? It's working fine and I've been testing it for the last few days without any issues. Would be great if it's merged.

@acacha
Copy link

acacha commented Dec 13, 2018

We also need this! Very useful @irazasyed

@vesper8 vesper8 mentioned this pull request Dec 13, 2018
@vesper8
Copy link

vesper8 commented Dec 13, 2018

will this allow to access the dashboard without a user at all? I mean, can I disable the authorize completely? This is what I do for Horizon/Telescope

I comment out the Authorize middleware completely and replace it by 'under-construction'

from https://github.com/larsjanssen6/underconstruction

This way I can protect my admin routes with a 4 digit pin and don't need to have a user system available at all on some of my apps

@irazasyed
Copy link
Contributor Author

@vesper8 Yes, You can do that.

@mpociot
Copy link
Member

mpociot commented Dec 17, 2018

Thank you for the PR. This was now part of another PR that just got merged.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants