-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Feature suggestion: Authentication #1151
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
Comments
This is not currently in the scope of this project. We do provide our users with a simple way of doing this over on our wiki (Code Snippets -> Restrict access to a handler (decorator)). |
That's my intention. Make a default decorator that will be included in the package. |
Alright in that case. |
Design proposal:
Comments appreciated. |
A few suggestions:
|
I like those suggestions. Filters for roles should be there, of course. I fail to understand the custom function for role mappings so far, what stops you from simply adding it to the dispatcher or modifying the existing mapping in there? Another thing: |
Maybe a function is not the best way to do it. What I meant is that there should be a way to ensure that your role mapping is always up-to-date. For example: |
Is there anyone that is working on this, or any good reason to keep this open? Will close after 2 weeks without respons |
I needed this (and a couple other things) myself so i wrote a lib for that. |
I wrote an authentication function to my private bots, to control who can use the bot.
I thought to add it to this project, but there are 2 problems:
The text was updated successfully, but these errors were encountered: