Skip to content
New issue

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

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(oidc-provider): add stack error on error mapping #2297

Open
wants to merge 1 commit into
base: production
Choose a base branch
from

Conversation

Romakita
Copy link
Collaborator

@Romakita Romakita commented Apr 1, 2023

No description provided.

@github-actions
Copy link

github-actions bot commented Apr 1, 2023

Benchmarks

  • Machine: linux x64 | 2 vCPUs | 6.8GB Mem
  • Node: v16.19.1
  • Run: Sat Apr 01 2023 10:10:59 GMT+0000 (Coordinated Universal Time)
  • Method: autocannon -c 100 -d 10 -p 10 localhost:3000 (two rounds; one to warm-up, one to measure)
Version Router Requests/s Latency Throughput/Mb
fastify 3.29.4 44425.6 22.05 7.92
nest-fastify 8.4.3 40921.6 23.98 7.30
koa 2.13.4 34639.2 28.48 6.18
fastify-injector 3.29.4 20115.2 49.21 4.59
express 4.18.1 8652.4 114.81 1.54
fastify-big-json 3.29.4 7873.0 125.89 90.56
tsed-koa 7.18.3 7759.8 127.74 6.78
nest 8.4.3 7570.4 130.84 1.82
express-injector 4.18.1 6882.8 144.05 1.57
tsed-express 7.18.3 6075.8 163.03 1.11
express-morgan 4.18.1 4962.9 199.65 0.88

Explanation

The benchmark shows a performance difference between the frameworks. We note that Ts.ED is often last. In fact, Ts.ED uses features useful to a production application which reduce its performance.

For example, Ts.ED initializes a sandbox (async_hook) for each request in order to work in an isolated context if necessary.
It also initializes the elements necessary for monitoring requests in a log manager.

All this at a necessary cost that reflects the reality of a production application ;)

@Romakita
Copy link
Collaborator Author

Romakita commented Sep 3, 2023

/rebase

@Romakita Romakita force-pushed the production branch 9 times, most recently from df09689 to eed91ec Compare September 4, 2023 07:42
@Romakita Romakita force-pushed the production branch 6 times, most recently from f37ccd1 to 5306644 Compare February 18, 2024 10:58
@Romakita Romakita force-pushed the production branch 4 times, most recently from b76e7c8 to 9230a2a Compare August 25, 2024 08:18
@Romakita Romakita force-pushed the production branch 5 times, most recently from f29ad02 to 22659f4 Compare December 1, 2024 09:34
@Romakita Romakita force-pushed the production branch 2 times, most recently from e7aa499 to 000a6b4 Compare December 5, 2024 20:03
@Romakita Romakita force-pushed the production branch 2 times, most recently from de8df3b to 474e2d8 Compare December 17, 2024 19:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant