search

joestubbs / tagent

0 stars 1 forks source link

Error handling for handlers #12

Closed joestubbs closed 2 years ago

joestubbs commented 2 years ago

The main change in this PR is to update the request handlers to all use TagentError, and to move TagentError to representations.

Additionally, this PR fixes the upload and download handlers that were not converted to the new actix-web v4 application state object, and it adds some logging and code clean up.

Finally, I added a Makefile to help me remember the commands I need to run. I don't know if Makefiles are "the Rust way" so if not we can do something else.

Let me know your thoughts on any of this!

waltermoreira commented 2 years ago

Hope you don't mind, I pushed a commit to fix that import that was giving us problem. I'll keep taking a look...