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

Predefined ACL rule "local". #12

Closed
wants to merge 1 commit into from
Closed

Conversation

jadahl
Copy link
Contributor

@jadahl jadahl commented Feb 27, 2011

Addition of another predefined ACL rule called "local" to match a user against either the virtual host of the match rule call, or any configured virtual host in case the rule is on the global scope.

@badlop
Copy link
Member

badlop commented Mar 3, 2011

I don't understand the practical difference between your predefined ACL rule "local" and the ACL rule included in the default ejabberd.cfg:
{acl, local, {user_regexp, ""}}.

Both have the same effect in practice, right? Otherwise, what's the benefit of using your ACL rule?

@jadahl
Copy link
Contributor Author

jadahl commented Mar 3, 2011

It's true that using the user regexp match rule would have the same effect, but it would be more intuitive to have this option as well. While not using anything based on the default configuration file, looking through the documentation for accurate ACL rule, I felt this option was missing.

goj referenced this pull request in goj/ejabberd Jan 13, 2012
Make ./bin/ejabberd more like the old ejabberdctl
@badlop badlop closed this Jan 29, 2015
hamano pushed a commit to hamano/ejabberd that referenced this pull request Sep 12, 2015
@lock
Copy link

lock bot commented Jun 10, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jun 10, 2019
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.

None yet

2 participants