Update python examples #529
Conversation
Update python example
| encrypted: | ||
| - column: data | ||
| data_type: bytes | ||
| # base64 bytes value |
There was a problem hiding this comment.
mention decoded value of base64 string to understand what we should expect: "test-data"
examples/python/example_with_zone.py
Outdated
| result = connection.execute( | ||
| # explicitly pass zone id before related data | ||
| select([cast(zone_id.encode('utf-8'), BYTEA), test_table])) | ||
| select([literal_column("'{}'".format(zone_id)), test_table])) |
There was a problem hiding this comment.
what about to avoid manual escaping values, use sqlalchemy's engine and specify type_ value for literal_column? I think that we can specify String type to apply escape rules for string values (according to TypeEngine documentation)?
if it will works, I prefer to use such approach. Because it will automatically work for mysql and all other DBs if we will add them.
There was a problem hiding this comment.
Sounds good, let me check it.
There was a problem hiding this comment.
I used sqlalchemy's literal, it performs escaping values inside. What do you think @Lagovas ?
Fixed after review
Lagovas
left a comment
Lagovas
left a comment
There was a problem hiding this comment.
lgtm if it works with literal. I remember that had some problems with that. But probably it was related with compiling queries with bound values.
# Conflicts: # examples/python/encryptor_config_with_zone.yaml
* zhars/update_python_examples Update python examples
Actualizing python examples.
encryptor_config_*with explicit table declaration to run with transparent encryption and be able to show type awareness functionalityzone_idfield to be able to match zone from the query but not from the table.write_datafunction with explicit zone_id literal - deletedcastoperator as it is not supported by Acra sql parser now.Checklist
with new changes