Skip to content

Commit 8e11c7c

Browse files
committed
Fixed some invalid html, and tweaked the html validation settings in the test crawler.
- Legacy-Id: 12066
1 parent cd2d8a5 commit 8e11c7c

3 files changed

Lines changed: 10 additions & 6 deletions

File tree

bin/test-crawl

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ def check_html_valid(url, response, args):
133133
key = re.sub("/draft-[a-z0-9-]+/", "/draft-foo/", key)
134134
key = re.sub("/group/[a-z0-9-]+/", "/group/foo/", key)
135135
key = re.sub("/ipr/search/.*", "/ipr/search/", key)
136-
key = re.sub("/meeting/[0-9]+/agenda/[0-9a-z]+/", "/meeting/nn/agenda/foo/", key)
136+
key = re.sub("/meeting/[-0-9a-z]+/agenda/[0-9a-z]+/", "/meeting/nn/agenda/foo/", key)
137137
key = re.sub("/release/[0-9dev.]+/", "/release/n.n.n/", key)
138138
key = re.sub("/rfc[0-9]+/", "/rfcnnnn/", key)
139139
key = re.sub("/rg/[a-z0-9-]+/", "/rg/foo/", key)
@@ -248,7 +248,9 @@ for url in initial_urls:
248248

249249
parser = html5lib.HTMLParser(strict=True)
250250

251-
validated_urls = {}
251+
# initialise validated_urls with some patterns we don't want to check,
252+
# because they aren't under our control, such as uploaded group agendas.
253+
validated_urls = {'/meeting/nn/agenda/foo/': True, }
252254

253255
doc_types = [ t.slug for t in DocTypeName.objects.all() ]
254256

ietf/templates/group/group_about.html

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@
3535
{% endif %}
3636
</tr>
3737

38+
<tr>
3839
<td></td>
3940
<th>State</th>
4041
<td>
@@ -43,6 +44,7 @@
4344
<div class="label label-info">In the process of being closed</div>
4445
{% endif %}
4546
</td>
47+
</tr>
4648

4749
{% if group.features.has_chartering_process %}
4850
<tr>

ietf/templates/release/release.html

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -56,21 +56,21 @@ <h2>Release list:</h2>
5656
{% for item in releases %}
5757
{% ifchanged %}
5858
</tbody>
59-
<thead>
59+
<tbody>
60+
<tr><td colspan="3"></td></tr>
6061
<tr class="year-row">
61-
<th><h3>{{ item.date|slice:"7:11" }}</h3></tth>
62+
<th class="h3">{{ item.date|slice:"7:11" }}</th>
6263
<th></th>
6364
<th></th>
6465
</tr>
65-
</thead>
66-
<tbody>
6766
{% endifchanged %}
6867
<tr class="version-row">
6968
<td class="version"><a href="/release/{{item.version}}/">{{ item.version }}</a></td>
7069
<td class="date">{{ item.date|slice:":11" }}</td>
7170
<td class="release-title">{{ item.title }}</td>
7271
</tr>
7372
{% endfor %}
73+
</tbody>
7474
</table>
7575
</div>
7676
{% endblock %}

0 commit comments

Comments
 (0)