Merge branch 'master' of git://github.com/ask/chishop

6 files changed, 102 insertions, 52 deletions

diff --git a/AUTHORS b/AUTHORS
index 71b6d81..9d005e6 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -1,3 +1,4 @@
 Ask Solem <askh@opera.com>
 Rune Halvorsen <runeh@opera.com>
 Russell Sim <russell.sim@gmail.com>
+Brian Rosner <brosner@gmail.com>
diff --git a/TODO b/TODO
index ae681f6..5d57eb7 100644
--- a/TODO
+++ b/TODO
@@ -1,12 +1,52 @@
-* Write a tutorial on how to set up the server, registering projects, and
-  how to upload releases.
+PyPI feature replication
+========================
+
 * Make it possible to register users via distutils.
-  (There should be a setting to turn this feature on/off).
-* Add a pretty, user-friendly web site interface in addition to
-  the API interface.
-* Maybe add a permission "can upload new release", so more than one
-  user can change the same project.
-* Should a project have co-owners?
+  There should be a setting to turn this feature on/off for private PyPIs.
+  [taken-by: sverrej]
+
+* Roles (co-owners/maintainers)
     - One possible solution:
             http://github.com/initcrash/django-object-permissions/tree
-            
+    I'm not sure what the difference between a co-owner and maintainer is,
+    maybe it's just a label.
+* Package author admin interface (submit, edit, view)
+* Search
+* Documentation upload
+* Ratings
+* Random Monty Python quotes :-)
+* Comments :-)
+
+Post-PyPI
+=========
+
+* PEP-381: Mirroring infrastructure for PyPI
+    [taken-by: jezdez]
+
+* API to submit test reports for smoke test bots. Like CPAN Testers.
+  Platform/version/matrix etc.
+
+* Different listings: Author listings, classifier listings, etc.
+
+* Search metadata
+
+* Automatic generation of Sphinx for modules (so you can view them directly
+on pypi, like CPAN), Module listing etc.
+
+* Listing of special files: README, LICENSE, Changefile/Changes, TODO,
+    MANIFEST.
+
+* Dependency graphs.
+
+* Package file browser (like CPAN)
+
+
+
+
+Documentation
+=============
+
+* Write a tutorial on how to set up the server, registering projects, and
+  how to upload releases.
+
+
diff --git a/chishop/settings.py b/chishop/settings.py
index 00cf06d..180a652 100644
--- a/chishop/settings.py
+++ b/chishop/settings.py
@@ -12,6 +12,7 @@ ADMINS = (
 # if you're sloppy.
 DJANGOPYPI_ALLOW_VERSION_OVERWRITE = False
 DJANGOPYPI_RELEASE_UPLOAD_TO = 'dists'
+LOCAL_DEVELOPMENT=True
 
 # change to False if you do not want Django's default server to serve static pages
 LOCAL_DEVELOPMENT = True
diff --git a/djangopypi/templates/djangopypi/pypi_show_links.html b/djangopypi/templates/djangopypi/pypi_show_links.html
index 83142b7..6b239d1 100644
--- a/djangopypi/templates/djangopypi/pypi_show_links.html
+++ b/djangopypi/templates/djangopypi/pypi_show_links.html
@@ -12,7 +12,7 @@
 <tbody>
     {% for release in releases %}
     <tr>
-        <td><a href="/{{ release.get_dl_url }}">{{ release.filename }}</a></td>
+        <td><a href="{{ release.get_dl_url }}">{{ release.filename }}</a></td>
         <td>{{ release.platform }}</td>
         <td>{{ release.type }}</td>
         <td>{{ release.pyversion }}</td>
diff --git a/djangopypi/utils.py b/djangopypi/utils.py
new file mode 100644
index 0000000..ba2013e
--- /dev/null
+++ b/djangopypi/utils.py
@@ -0,0 +1,38 @@
+import sys
+import traceback
+
+from django.core.files.uploadedfile import SimpleUploadedFile
+from django.utils.datastructures import MultiValueDict
+
+
+def transmute(f):
+    if hasattr(f, "filename") and f.filename:
+        v = SimpleUploadedFile(f.filename, f.value, f.type)
+    else:
+        v = f.value.decode("utf-8")
+    return v
+
+
+def decode_fs(fs):
+    POST, FILES = {}, {}
+    for k in fs.keys():
+        v = transmute(fs[k])
+        if isinstance(v, SimpleUploadedFile):
+            FILES[k] = [v]
+        else:
+            # Distutils sends UNKNOWN for empty fields (e.g platform)
+            # [russell.sim@gmail.com]
+            if v == "UNKNOWN":
+                v = None
+            POST[k] = [v]
+    return MultiValueDict(POST), MultiValueDict(FILES)
+
+
+def debug(func):
+    # @debug is handy when debugging distutils requests
+    def _wrapped(*args, **kwargs):
+        try:
+            return func(*args, **kwargs)
+        except:
+            traceback.print_exception(*sys.exc_info())
+    return _wrapped
\ No newline at end of file
diff --git a/djangopypi/views.py b/djangopypi/views.py
index 90929a1..784651b 100644
--- a/djangopypi/views.py
+++ b/djangopypi/views.py
@@ -30,8 +30,14 @@ POSSIBILITY OF SUCH DAMAGE.
 
 """
 
+import cgi
 import os
 
+try:
+    from cStringIO import StringIO
+except ImportError:
+    from StringIO import StringIO
+
 from django.conf import settings
 from django.http import Http404, HttpResponse, HttpResponseBadRequest
 from django.http import QueryDict, HttpResponseForbidden
@@ -45,53 +51,17 @@ from django.core.files.uploadedfile import SimpleUploadedFile
 from django.contrib.auth import authenticate, login
 from djangopypi.http import HttpResponseNotImplemented
 from djangopypi.http import HttpResponseUnauthorized
+from djangopypi.utils import decode_fs
 
 
 ALREADY_EXISTS_FMT = _("""A file named "%s" already exists for %s. To fix """
                      + "problems with that you should create a new release.")
 
 
-def parse_weird_post_data(raw_post_data):
-    """ For some reason Django can't parse the HTTP POST data
-    sent by ``distutils`` register/upload commands.
-
-    This parser should be able to so, and returns a
-    :class:`django.utils.datastructures.MultiValueDict`
-    as you would expect from a regular ``request.POST`` object.
-    """
-    # If anyone knows any better way to do this, you're welcome
-    # to give me a solid beating. [askh@opera.com]
-    sep = raw_post_data.splitlines()[1]
-    items = raw_post_data.split(sep)
-    post_data = {}
-    files = {}
-    for part in [e for e in items if not e.isspace()]:
-        item = part.splitlines()
-        if len(item) < 2: continue
-        header = item[1].replace("Content-Disposition: form-data; ", "")
-        kvpairs = header.split(";")
-        headers = {}
-        for kvpair in kvpairs:
-            if not kvpair: continue
-            key, value = kvpair.split("=")
-            headers[key] = value.strip('"')
-        if not "name" in headers: continue
-        content = part[len("\n".join(item[0:2]))+2:len(part)-1]
-        if "filename" in headers:
-            file = SimpleUploadedFile(headers["filename"], content,
-                    content_type="application/gzip")
-            files["distribution"] = [file]
-        elif headers["name"] in post_data:
-            post_data[headers["name"]].append(content)
-        else:
-            # Distutils sends UNKNOWN for empty fields (e.g platform)
-            # [russell.sim@gmail.com]
-            if content == 'UNKNOWN':
-                post_data[headers["name"]] = [None]
-            else:
-                post_data[headers["name"]] = [content]
-
-    return MultiValueDict(post_data), MultiValueDict(files)
+def parse_distutils_request(request):
+    fp = StringIO(request.raw_post_data)
+    fs = cgi.FieldStorage(fp=fp, environ=request.META)
+    return decode_fs(fs)
 
 
 def login_basic_auth(request):
@@ -181,7 +151,7 @@ ACTIONS = {
 
 def simple(request, template_name="djangopypi/simple.html"):
     if request.method == "POST":
-        post_data, files = parse_weird_post_data(request.raw_post_data)
+        post_data, files = parse_distutils_request(request.raw_post_data)
         action_name = post_data.get(":action")
         if action_name not in ACTIONS:
             return HttpResponseNotImplemented(