Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃帀 Add Fastify as additional web layer #160

Open
goldbergyoni opened this issue Jul 6, 2022 · 3 comments
Open

馃帀 Add Fastify as additional web layer #160

goldbergyoni opened this issue Jul 6, 2022 · 3 comments

Comments

@goldbergyoni
Copy link
Contributor

馃幆 Goal: - Allowing the user the choice between various technologies is at the heart of Practica. Fastify is a super-popular and great framework (a candidate to serve as our recommended web layer). We should add it as an alternative web layer to express, it will be located in a sibling folder and the code generator while stitching the solution will copy OR express OR Fastify as the web layer. This is agnostic to the domain and data-access layer which will always be the same

馃 Things to consider:

  • Fastify serves as our web layer, hence it's responsible for JWT validation, and other web-related tasks. With that in mind, our existing libraries might need to change to support Fastify or maybe we should add dedicated new libraries?
  • Some existing functionality is built inside Fastify and in that case we should use Fastify without abusing it. For example, Fastify has a logger built-in and a validation framework
@kalitas
Copy link
Collaborator

kalitas commented Jul 6, 2022

on it

@goldbergyoni
Copy link
Contributor Author

@DavidBK and @kalitas Happy that you're handling this strategic and not-so-easy workstream

I would guess that the status is @DavidBK working on the Fastify thing and @kalitas on the code generator, right?

Leme know whether you need anything from my part to push it?

@kalitas
Copy link
Collaborator

kalitas commented Jul 21, 2022

@goldbergyoni yes. @DavidBK if you can create a PR from your fork to here, it would be great, then we can continue working on it and finish it together

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants