[buildinfo.debian.net] 02/04: Factor out "Format:" parsing.

[Chris Lamb]

This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository buildinfo.debian.net.

commit e3b7fb16d1e19e4d4cf73bbd0e6d7b1313838ec2
Author: Chris Lamb <lamby at debian.org>
Date:   Sat Mar 25 22:12:40 2017 +0000

    Factor out "Format:" parsing.
    
    Signed-off-by: Chris Lamb <lamby at debian.org>
---
 bidb/api/utils.py | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/bidb/api/utils.py b/bidb/api/utils.py
index 208a180..2ef4e8a 100644
--- a/bidb/api/utils.py
+++ b/bidb/api/utils.py
@@ -77,8 +77,13 @@ def parse_submission(request):
         except KeyError:
             raise InvalidSubmission("Missing required field: {}".format(field))
 
-    if data.get('Format') not in SUPPORTED_FORMATS:
-        raise InvalidSubmission("Only {} format versins are supported".format(
+    try:
+        version = data.get('Format')
+    except KeyError:
+        raise InvalidSubmission("Missing 'Format' header")
+
+    if version not in SUPPORTED_FORMATS:
+        raise InvalidSubmission("Only {} format versions are supported".format(
             ', '.join(sorted(SUPPORTED_FORMATS)),
         ))
 
@@ -87,7 +92,7 @@ def parse_submission(request):
 
         source=get_or_create(Source, 'Source'),
         architecture=get_or_create(Architecture, 'Architecture'),
-        version=data['Version'],
+        version=version,
 
         build_path=data.get('Build-Path', ''),
         build_date=parse(data.get('Build-Date', '')),

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/reproducible/buildinfo.debian.net.git