add executor to validate_payload

[drc38]

Addresses issue #647

[jainmohit2001]

Hi @drc38, the changes look good. I believe the functionality will not break and it is backwards compatible as well.

Here's another thought: Instead of using run_in_executor to run the blocking validate_payload function call, what if we convert the validate_payload function into a pure async function with all I/O calls being non-blocking.

[drc38]

Hi @drc38, the changes look good. I believe the functionality will not break and it is backwards compatible as well.

Here's another thought: Instead of using run_in_executor to run the blocking validate_payload function call, what if we convert the validate_payload function into a pure async function with all I/O calls being non-blocking.

That would result in quite a departure from the current structure where most functions are not async. Another approach is to use https://github.com/Tinche/aiofiles for the file open, assuming you're happy to import aiofiles as part of the project.

[drc38]

That would result in quite a departure from the current structure where most functions are not async. Another approach is to use https://github.com/Tinche/aiofiles for the file open, assuming you're happy to import aiofiles as part of the project.

Actually using aiofiles would still require a rewrite to async. I think the proposed approach is the best for now.