msgid is stored as text type about pybitmessage HOT 5 OPEN

This is true, but there are potential problems with upgrades. That's why the sql thread should be refactored first and added tests. These are the PRs that tried that but they are too big and need cleaning up: #1794 #1999 #2150

from pybitmessage.

The type of values whose declared type is blob:

• inbox.msgid : text
• inbox.sighash : text
• inventory.hash : blob
• inventory.payload : blob
• inventory.tag : blob
• objectprocessorqueue.data : (empty)
• pubkeys.transmitdata : text
• sent.msgid : text
• sent.toripe : text
• sent.ackdata : text
• settings.key : text
• settings.value : text, integer

from pybitmessage.

It actually isn't consistent across versions, there may be older databases which are different. That's why we need tests for this, even more for the data type change.

from pybitmessage.

How about this:

• Change all code at once to store all blob data as blob by using sqlite3.Binary(). Then,
• Search any blob data:
  • First, try SELECT by blob keys using sqlite3.Binary(key).
  • When there is no match found, as a fallback, try SELECT by text key using CAST(? AS TEXT) in SQLite sintax.

I think this should be compatible to any past versions and future updates, including migrations to Python3.

from pybitmessage.

I have implemented a quick workaround: #2248

from pybitmessage.