nik/FreedomBox
1
0
Fork 0
mirror of https://github.com/freedombox/FreedomBox.git synced 2026-03-11 09:04:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

users: Add list of restricted usernammes

Browse Source 
List is derived from optional services installed.
This commit is contained in:
James Valleroy 2016-07-14 08:13:41 -04:00 committed by Sunil Mohan Adapa
parent 3a69958165
commit c8c6bc377b
No known key found for this signature in database
GPG Key ID: 36C361440C9BC971
2 changed files with 24 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
plinth/modules/first_boot/forms.py
View File

@ -35,7 +35,7 @@ from plinth import cfg
from plinth.errors import ActionError, DomainRegistrationError
from plinth.modules.pagekite.utils import PREDEFINED_SERVICES, run
from plinth.modules.security import set_restricted_access
from plinth.modules.users.forms import GROUP_CHOICES
from plinth.modules.users.forms import GROUP_CHOICES, RESTRICTED_USERNAMES
from plinth.utils import format_lazy
logger = logging.getLogger(__name__)
@ -55,7 +55,8 @@ class State1Form(auth.forms.UserCreationForm):
# Exit code 0 means that the username is already in use.
raise ValidationError(_('Username is reserved'))
except subprocess.CalledProcessError:
pass
if username in RESTRICTED_USERNAMES:
raise ValidationError(_('Username is reserved'))
return super().clean()

23
plinth/modules/users/forms.py
View File

@ -27,6 +27,23 @@ from django.utils.translation import ugettext as _, ugettext_lazy
from plinth import actions
from plinth.errors import ActionError
# Usernames used by optional services (that might not be installed yet).
RESTRICTED_USERNAMES = [
'debian-deluged',
'Debian-minetest',
'debian-tor',
'debian-transmission',
'ejabberd',
'ez-ipupd',
'monkeysphere',
'mumble-server',
'node-restore',
'quasselcore',
'radicale',
'repro',
'privoxy',
]
GROUP_CHOICES = (
('admin', _('admin')),
('wiki', _('wiki')),
@ -66,7 +83,8 @@ class CreateUserForm(UserCreationForm):
# Exit code 0 means that the username is already in use.
raise ValidationError(_('Username is reserved'))
except subprocess.CalledProcessError:
pass
if username in RESTRICTED_USERNAMES:
raise ValidationError(_('Username is reserved'))
return super().clean()
@ -139,7 +157,8 @@ class UserUpdateForm(forms.ModelForm):
# Exit code 0 means that the username is already in use.
raise ValidationError(_('Username is reserved'))
except subprocess.CalledProcessError:
pass
if username in RESTRICTED_USERNAMES:
raise ValidationError(_('Username is reserved'))
return super().clean()