Check validity of repo names
BUG:FIXED:5906
This commit is contained in:
@@ -10,6 +10,7 @@
|
|||||||
# Please read the COPYING file.
|
# Please read the COPYING file.
|
||||||
|
|
||||||
import os
|
import os
|
||||||
|
import re
|
||||||
import logging
|
import logging
|
||||||
import logging.handlers
|
import logging.handlers
|
||||||
|
|
||||||
@@ -580,6 +581,8 @@ def index(dirs=None, output='pisi-index.xml', skip_sources=False, skip_signing=F
|
|||||||
ctx.ui.info(_('* Index file written'))
|
ctx.ui.info(_('* Index file written'))
|
||||||
|
|
||||||
def add_repo(name, indexuri, at = None):
|
def add_repo(name, indexuri, at = None):
|
||||||
|
if not re.match("^[a-zA-Z0-9\\-\\_]*$", name):
|
||||||
|
raise pisi.Error(_('Not a valid repo name.'))
|
||||||
repodb = pisi.db.repodb.RepoDB()
|
repodb = pisi.db.repodb.RepoDB()
|
||||||
if repodb.has_repo(name):
|
if repodb.has_repo(name):
|
||||||
raise pisi.Error(_('Repo %s already present.') % name)
|
raise pisi.Error(_('Repo %s already present.') % name)
|
||||||
|
|||||||
Reference in New Issue
Block a user