Vanilla-OS / chronos-frontend

A frontend in Vue.js for the Chronos documentation server.
https://docs.vanillaos.org
GNU Affero General Public License v3.0
6 stars 4 forks source link

Implemented Fastify server for performance enhancements. #61

Closed 7Silva closed 1 month ago

7Silva commented 4 months ago

Hello Vanilla-OS team,

In this pull request, I implemented Fastify and removed Express. I made this change because Fastify is much more optimized for requests than Express. Below, I will leave some benchmark screenshots, but you can also see others and do your own tests. The main changes include:

I did not change any logic behind the system, I just made this replacement and added some very useful options to Fastify, such as:

I created an index file that already creates the Chronos file and th en starts the server. I thought about doing a check in the index to see if there is an existing Chronos configuration and, if there is, not to generate it again. However, I don't know if this would be useful for you, so I decided not to do it.

Overall, these are the changes that were made. I hope they are useful and I look forward to your feedback.

Express Server: Express

Fastify Server: Fastify

mirkobrombin commented 4 months ago

Somehow this issue escaped from my radar. I'll take a look at it asap.

mirkobrombin commented 4 months ago

Looks good to me, need to be updated, I can take care of it.

kbdharun commented 2 months ago

Looks good to me, need to be updated, I can take care of it.

Fixed the failing build (the package files weren't in sync, updated it with npm install and tested the changes locally). Rebase and merge seem to be blocked (can we use Squash & merge here @mirkobrombin? Or is there any other changes that you want to implement?)

mirkobrombin commented 2 months ago

Squash and merge is fine