comparison mercurial/hgweb/hgweb_mod.py @ 25777:1c2a8db33b8f

hgweb: allow symbolic revisions with forward slashes in urls It's possible to have a branch/tag/bookmark with all kinds of special characters, such as {}/\!?. While not very conveniently, symbolic revisions with such characters work from command line if user correctly quotes the characters. These characters also work in hgweb, when they are properly encoded, with one exception: '/' (forward slash, urlencoded as '%2F'), which was getting decoded before hgweb could parse it as a part of PATH_INFO. Because of that, hgweb was seeing it as any other forward slash, that is, as just another url parts separator. For example, if user wanted to see the content of dir/file at bookmark 'feature/eggs', url could be: '/file/feature%2Feggs/dir/file'. But hgweb tried to find a revision 'feature' and get contents of 'eggs/dir/file'. To fix this, let's assume forward slashes are doubly-urlencoded (%252F), so CGI/WSGI server decodes it into %2F. Then we can decode %2F in the revision part of the url into an actual '/' character. Making hgweb produce such urls will be done in the next 2 patches.
author Anton Shestakov <av6@dwimlabs.net>
date Sun, 12 Jul 2015 16:06:57 +0800
parents bd2e171d023b
children 1a45e49a6bed 9df8c729e2e7
comparison
equal deleted inserted replaced
25776:809c3e4a9e31 25777:1c2a8db33b8f
233 233
234 if cmd == 'static': 234 if cmd == 'static':
235 req.form['file'] = ['/'.join(args)] 235 req.form['file'] = ['/'.join(args)]
236 else: 236 else:
237 if args and args[0]: 237 if args and args[0]:
238 node = args.pop(0) 238 node = args.pop(0).replace('%2F', '/')
239 req.form['node'] = [node] 239 req.form['node'] = [node]
240 if args: 240 if args:
241 req.form['file'] = args 241 req.form['file'] = args
242 242
243 ua = req.env.get('HTTP_USER_AGENT', '') 243 ua = req.env.get('HTTP_USER_AGENT', '')