mirror of
https://github.com/hanzoai/tasks.git
synced 2026-10-30 14:56:19 +00:00
David ReissandGitHub
6bfd7fec0c
## What changed? Remove various utf-8 validation calls that are no longer needed. ## Why? For a long time now, all proto marshaling and unmarshaling has validated strings are valid utf-8, so all data coming in through grpc must be valid utf-8. We don't need to validate specific fields anymore. ## How did you test it? - [ ] built - [ ] run locally and tested manually - [x] covered by existing tests - [ ] added new unit test(s) - [ ] added new functional test(s) ## Potential risks Any data that does _not_ come through grpc, e.g. persistence, may still need to be validated. Most data in persistence is in protos so that's fine. We may want to add explicit validation for string fields around calls to gocql, for example, which doesn't appear to validate strings for utf-8.