From 51c125a06e087dc92a861a47ed8b3bc190c35428 Mon Sep 17 00:00:00 2001 From: Patrick Uiterwijk Date: Sun, 4 Jun 2017 01:17:44 +0200 Subject: [PATCH] Make sure to fix encoding all RPM Headers The fix in #403 left some RPM Headers unfixed, most notably changelog* (because those are lists). This patch makes use of fixEncodingRecurse to make sure all heades get encoded correctly. Signed-off-by: Patrick Uiterwijk --- hub/kojihub.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/hub/kojihub.py b/hub/kojihub.py index 947f60dd..d3000384 100644 --- a/hub/kojihub.py +++ b/hub/kojihub.py @@ -9883,10 +9883,7 @@ class RootExports(object): raise koji.GenericError('either rpmID or taskID and filepath must be specified') headers = koji.get_header_fields(rpm_path, headers) - for key, value in headers.items(): - if isinstance(value, six.string_types): - headers[key] = koji.fixEncoding(value, remove_nonprintable=True) - return headers + return koji.fixEncodingRecurse(headers, remove_nonprintable=True) queryRPMSigs = staticmethod(query_rpm_sigs)