You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Oct 13, 2020. It is now read-only.
Thanks for this library, really like what you've done @tommelo 👍
I was wondering whether you've worked on a version that supports async/await? If not, do you think the changes would be around making execute and before async?
Actually, this lib needs a huge refactor, everything was done with a tight deadline at the company.
I'll look forward to make some improvements(including async/await).
If you have any other suggestion or If you want to contribute with code, design, tests and etc, just let me know, that would be great!
When you say huge refactor do you mean a complete re-write? If you could list out some items that are in your head we can maybe come up with a little roadmap of improvements 😉
Yes, a complete re-write!
I guess we could start trying to figure out a better design for the following features:
Global Exception Handling;
Request Serialization based on HTTP Header "Accept" (XML and JSON);
Request Deserialization based on HTTP Header "Content-Type"(x-www-form-url-encoded, application/json and etc);
Built in Request Validation(Joi or somenthing else).
Request Route? (for example, if the user wants to handle the methods GET, POST, PUT, DELETE for the endpoint /users using just one Lambda Function).
Then, when we get this done, it would be great to have a cli tool to create project "boilerplates" or "templates" (controllers, validators, cloudformation templates and etc).
Let me know what you think!
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Hi there,
Thanks for this library, really like what you've done @tommelo 👍
I was wondering whether you've worked on a version that supports async/await? If not, do you think the changes would be around making
execute
andbefore
async?aws-wave/lib/handler/abstract-request-handler.js
Line 76 in fa198c6
Thanks,
Andre
The text was updated successfully, but these errors were encountered: