koji

Clone

#2944 decode protonmsg props field when loading from db
Merged 2 years ago by tkopecek. Opened 2 years ago by mikem.
mikem/koji protonmsg-decode-props  into  master

file modified
+5 
@@ -348,6 +348,11 @@ 
 

                                 columns=('id', 'address', 'props', 'body'),
 

                                 opts={'order': 'id', 'limit': limit})
 

          msgs = list(query.execute())
 

+         for msg in msgs:
 

+             try:
 

+                 msg['props'] = json.loads(msg['props'])
 

+             except Exception:
 

+                 LOG.error('Unable to load message from db: %r', msg)
 

          if not msgs:
 

              return
 

          if CONFIG.getboolean('broker', 'test_mode', fallback=False):

props is encoded as json on insertion, so we should decode it when we read it back

:thumbsup:

Commit 34d1c8e fixes this pull-request

Pull-Request has been merged by tkopecek
2 years ago

Merged by mistake. Reverted in 0e2ebb4
Metadata
None
No Tags
Changes Summary 1
+5 -0
file changed
plugins/hub/protonmsg.py
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.