From 555a69b1f59e650c28e77952fc4d3985e63e0c4c Mon Sep 17 00:00:00 2001
From: Thierry Parmentelat <thierry.parmentelat@inria.fr>
Date: Fri, 5 Jun 2015 10:10:29 +0200
Subject: [PATCH] remove printing 'name' in low-level record utilities, as
 UTF-8 values can create serious errors on the server side if ill-configured

---
 sfa/storage/model.py | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/sfa/storage/model.py b/sfa/storage/model.py
index 3b53517a..347012d7 100644
--- a/sfa/storage/model.py
+++ b/sfa/storage/model.py
@@ -110,7 +110,9 @@ class RegRecord (Base, AlchemyObj):
         result="<Record id=%s, type=%s, hrn=%s, authority=%s" % \
                 (self.record_id, self.type, self.hrn, self.authority)
 #        for extra in ('pointer', 'email', 'name'):
-        for extra in ('email', 'name'):
+#        for extra in ('email', 'name'):
+# displaying names at this point it too dangerous, because of unicode
+        for extra in ('email'):
             if hasattr(self, extra):
                 result += " {}={},".format(extra, getattr(self, extra))
         # skip the uniform '--- BEGIN CERTIFICATE --' stuff
@@ -208,7 +210,8 @@ class RegAuthority (RegRecord):
     # no proper data yet, just hack the typename
     def __repr__ (self):
         result = RegRecord.__repr__(self).replace("Record", "Authority")
-        result.replace(">", " name={}>".format(self.name))
+# here again trying to display names that can be utf8 is too dangerous        
+#        result.replace(">", " name={}>".format(self.name))
         return result
 
     def update_pis (self, pi_hrns, dbsession):
-- 
2.43.0