Skip to content

Commit

Permalink
Proxy, minor regression in authentication. contraints are always chec…
Browse files Browse the repository at this point in the history
…ked now, which should only apply for local. related to #3250

(cherry picked from commit 54551b0)
(cherry picked from commit e8dbda3)
  • Loading branch information
AdSchellevis authored and fichtner committed May 24, 2019
1 parent 5213f74 commit 450ff5b
Showing 1 changed file with 6 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,11 @@ public function getUserName()
public function checkConstraints()
{
// XXX: replace with group membership in 19.7
return (new ACL())->hasPrivilege($this->getUserName(), 'user-proxy-auth');
if (in_array('Local Database', $this->supportedAuthenticators())) {
return (new ACL())->hasPrivilege($this->getUserName(), 'user-proxy-auth');
} else {
// XXX non local auth, constraints not used. remove with https://github.com/opnsense/core/issues/3250
return true;
}
}
}

0 comments on commit 450ff5b

Please sign in to comment.