Page Menu
Home
Software Heritage
Search
Configure Global Search
Log In
Files
F9336743
D1290.id4074.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Award Token
Flag For Later
Size
5 KB
Subscribers
None
D1290.id4074.diff
View Options
diff --git a/swh/web/browse/views/directory.py b/swh/web/browse/views/directory.py
--- a/swh/web/browse/views/directory.py
+++ b/swh/web/browse/views/directory.py
@@ -21,8 +21,8 @@
from swh.web.browse.browseurls import browse_route
-@browse_route(r'directory/(?P<sha1_git>[0-9a-f]+)/',
- r'directory/(?P<sha1_git>[0-9a-f]+)/(?P<path>.+)/',
+@browse_route(r'directory/(?P<sha1_git>[0-9a-fA-F]+)/',
+ r'directory/(?P<sha1_git>[0-9a-fA-F]+)/(?P<path>.+)/',
view_name='browse-directory')
def directory_browse(request, sha1_git, path=None):
"""Django view for browsing the content of a directory identified
@@ -30,6 +30,7 @@
The url that points to it is :http:get:`/browse/directory/(sha1_git)/[(path)/]`
""" # noqa
+ sha1_git = sha1_git.lower()
root_sha1_git = sha1_git
try:
if path:
diff --git a/swh/web/browse/views/release.py b/swh/web/browse/views/release.py
--- a/swh/web/browse/views/release.py
+++ b/swh/web/browse/views/release.py
@@ -18,7 +18,7 @@
)
-@browse_route(r'release/(?P<sha1_git>[0-9a-f]+)/',
+@browse_route(r'release/(?P<sha1_git>[0-9a-fA-F]+)/',
view_name='browse-release')
def release_browse(request, sha1_git):
"""
@@ -27,6 +27,7 @@
The url that points to it is :http:get:`/browse/release/(sha1_git)/`.
"""
+ sha1_git = sha1_git.lower()
try:
release = service.lookup_release(sha1_git)
snapshot_context = None
diff --git a/swh/web/browse/views/snapshot.py b/swh/web/browse/views/snapshot.py
--- a/swh/web/browse/views/snapshot.py
+++ b/swh/web/browse/views/snapshot.py
@@ -16,21 +16,23 @@
)
-@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-f]+)/',
+@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-fA-F]+)/',
view_name='browse-snapshot')
def snapshot_browse(request, snapshot_id):
"""Django view for browsing the content of a snapshot.
The url that points to it is :http:get:`/browse/snapshot/(snapshot_id)/`
"""
+ snapshot_id = snapshot_id.lower()
browse_snapshot_url = reverse('browse-snapshot-directory',
url_args={'snapshot_id': snapshot_id},
query_params=request.GET)
return redirect(browse_snapshot_url)
-@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-f]+)/directory/',
- r'snapshot/(?P<snapshot_id>[0-9a-f]+)/directory/(?P<path>.+)/',
+@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-fA-F]+)/directory/',
+ r'''snapshot/(?P<snapshot_id>[0-9a-fA-F]+
+ )/directory/(?P<path>.+)/''',
view_name='browse-snapshot-directory')
def snapshot_directory_browse(request, snapshot_id, path=None):
"""Django view for browsing the content of a directory collected
@@ -38,6 +40,7 @@
The url that points to it is :http:get:`/browse/snapshot/(snapshot_id)/directory/[(path)/]`
""" # noqa
+ snapshot_id = snapshot_id.lower()
origin_type = request.GET.get('origin_type', None)
origin_url = request.GET.get('origin_url', None)
if not origin_url:
@@ -47,7 +50,7 @@
origin_url=origin_url)
-@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-f]+)/content/(?P<path>.+)/',
+@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-fA-F]+)/content/(?P<path>.+)/',
view_name='browse-snapshot-content')
def snapshot_content_browse(request, snapshot_id, path):
"""Django view that produces an HTML display of a content
@@ -55,10 +58,11 @@
The url that points to it is :http:get:`/browse/snapshot/(snapshot_id)/content/(path)/`
""" # noqa
+ snapshot_id = snapshot_id.lower()
return browse_snapshot_content(request, snapshot_id=snapshot_id, path=path)
-@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-f]+)/log/',
+@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-fA-F]+)/log/',
view_name='browse-snapshot-log')
def snapshot_log_browse(request, snapshot_id):
"""Django view that produces an HTML display of revisions history (aka
@@ -66,10 +70,11 @@
The url that points to it is :http:get:`/browse/snapshot/(snapshot_id)/log/`
""" # noqa
+ snapshot_id = snapshot_id.lower()
return browse_snapshot_log(request, snapshot_id=snapshot_id)
-@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-f]+)/branches/',
+@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-fA-F]+)/branches/',
view_name='browse-snapshot-branches')
def snapshot_branches_browse(request, snapshot_id):
"""Django view that produces an HTML display of the list of releases
@@ -77,10 +82,11 @@
The url that points to it is :http:get:`/browse/snapshot/(snapshot_id)/branches/`
""" # noqa
+ snapshot_id = snapshot_id.lower()
return browse_snapshot_branches(request, snapshot_id=snapshot_id)
-@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-f]+)/releases/',
+@browse_route(r'snapshot/(?P<snapshot_id>[0-9a-fA-F]+)/releases/',
view_name='browse-snapshot-releases')
def snapshot_releases_browse(request, snapshot_id):
"""Django view that produces an HTML display of the list of releases
@@ -88,4 +94,5 @@
The url that points to it is :http:get:`/browse/snapshot/(snapshot_id)/releases/`
""" # noqa
+ snapshot_id = snapshot_id.lower()
return browse_snapshot_releases(request, snapshot_id=snapshot_id)
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Jul 3 2025, 7:42 AM (10 w, 6 d ago)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
3222988
Attached To
D1290: Add support for uppercase sha1 url arguments
Event Timeline
Log In to Comment