This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	durin42	D5075 tests: accept slightly different zip file in Python 3
Closed	durin42	D5074 webcommands: use stringutil.pprint() to repr invalid archive types
Closed	durin42	D5073 archival: don't try and fsdecode non-{bytes,str} objects
Closed	durin42	D5072 tests: fix last failure in test-tools.t

Diff 12057

	No template is used for this handler. Raw, binary content is generated.			No template is used for this handler. Raw, binary content is generated.
	"""			"""

	type_ = web.req.qsparams.get('type')			type_ = web.req.qsparams.get('type')
	allowed = web.configlist("web", "allow-archive")			allowed = web.configlist("web", "allow-archive")
	key = web.req.qsparams['node']			key = web.req.qsparams['node']

	if type_ not in webutil.archivespecs:			if type_ not in webutil.archivespecs:
	msg = 'Unsupported archive type: %s' % type_			msg = 'Unsupported archive type: %s' % stringutil.pprint(type_)
	raise ErrorResponse(HTTP_NOT_FOUND, msg)			raise ErrorResponse(HTTP_NOT_FOUND, msg)

	if not ((type_ in allowed or			if not ((type_ in allowed or
	web.configbool("web", "allow" + type_))):			web.configbool("web", "allow" + type_))):
	msg = 'Archive type not allowed: %s' % type_			msg = 'Archive type not allowed: %s' % type_
	raise ErrorResponse(HTTP_FORBIDDEN, msg)			raise ErrorResponse(HTTP_FORBIDDEN, msg)

	reponame = re.sub(br"\W+", "-", os.path.basename(web.reponame))			reponame = re.sub(br"\W+", "-", os.path.basename(web.reponame))