modules/fm-orchestrator
1
0
Fork 0
mirror of https://pagure.io/fm-orchestrator.git synced 2026-04-13 16:59:52 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Use better way of checking for forbidden columns

Browse Source
This commit is contained in:
Jakub Kadlčík
2017-02-23 20:17:33 +01:00
parent 5a65f50b52
commit 62e26ba652
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
module_build_service/views.py
View File

@@ -124,7 +124,7 @@ class ModuleBuildAPI(MethodView):
log.error("The submitted scmurl %r is not valid" % url)
raise Unauthorized("The submitted scmurl %s is not valid" % url)
forbidden_params = [k for k in r if not hasattr(models.ModuleBuild, k)]
forbidden_params = [k for k in r if k not in models.ModuleBuild.__table__.columns]
if forbidden_params:
raise ValidationError('The request contains unspecified parameters: {}'.format(", ".join(forbidden_params)))
@@ -135,7 +135,7 @@ class ModuleBuildAPI(MethodView):
if not conf.yaml_submit_allowed:
raise Unauthorized("YAML submission is not enabled")
forbidden_params = [k for k in request.form if not hasattr(models.ModuleBuild, k)]
forbidden_params = [k for k in request.form if k not in models.ModuleBuild.__table__.columns]
if forbidden_params:
raise ValidationError('The request contains unspecified parameters: {}'.format(", ".join(forbidden_params)))