Point bibxml3 to I-D archive so that all version numbers are handled properly, and perform more robust revision suffix cleanup. Fixes #3188.

- Legacy-Id: 18905

ietf/doc/views_doc.py

@@ -832,8 +832,13 @@ def document_history(request, name):
 
 
 def document_bibtex(request, name, rev=None):
-    # If URL_REGEXPS put trailing digits in rev, they must be two digits
+    # Make sure URL_REGEXPS did not grab too much for the rev number
     if rev != None and len(rev) != 2:
+        mo = re.search(r"^(?P<m>[0-9]{1,2})-(?P<n>[0-9]{2})$", rev)
+        if mo:
+            name = name+"-"+mo.group(1)
+            rev = mo.group(2)
+        else:
             name = name+"-"+rev
             rev = None
 
@@ -875,9 +880,6 @@ def document_bibxml_ref(request, name, rev=None):
         raise Http404()
     if not name.startswith('draft-'):
         name = 'draft-'+name
-    if rev != None and len(rev) != 2:
-        name = name+"-"+rev
-        rev = None
     return document_bibxml(request, name, rev=rev)
     
 def document_bibxml(request, name, rev=None):
@@ -886,8 +888,13 @@ def document_bibxml(request, name, rev=None):
     if re.search(r'^rfc\d+$', name):
         raise Http404()
 
-    # If URL_REGEXPS put trailing digits in rev, they must be two digits
+    # Make sure URL_REGEXPS did not grab too much for the rev number
     if rev != None and len(rev) != 2:
+        mo = re.search(r"^(?P<m>[0-9]{1,2})-(?P<n>[0-9]{2})$", rev)
+        if mo:
+            name = name+"-"+mo.group(1)
+            rev = mo.group(2)
+        else:
             name = name+"-"+rev
             rev = None
 

ietf/templates/doc/bibxml.xml

@@ -16,5 +16,5 @@
    </front>
    <seriesInfo name="Internet-Draft" value="{{name}}-{{doc.rev}}" />
    {% if doi %}<seriesInfo name="DOI" value="{{doi}}" />
-   {% endif %}<format type="TXT" target="https://www.ietf.org/internet-drafts/{{name}}-{{doc.rev}}.txt" />
+   {% endif %}<format type="TXT" target="https://www.ietf.org/archive/id/{{name}}-{{doc.rev}}.txt" />
 </reference>