Switch server dependency from Express to Fastify #1638
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🦋 Changeset detectedLatest commit: 0593cf7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 168 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Is there an easy way to compare metrics of |
justinkaseman
commented
May 4, 2022
justinkaseman
commented
May 4, 2022
justinkaseman
commented
May 4, 2022
justinkaseman
commented
May 4, 2022
justinkaseman
commented
May 4, 2022
|
@bogos we will need a changeset here too. I'd call it a minor version bump since there are no breaking changes to the end user. |
|
Need to watch out for #1843. Anything added in that PR needs to be reflected in this PR. |
…ed customErrorMessage
alejoberardino
approved these changes
May 12, 2022
boxhock
approved these changes
May 12, 2022
mjk90
pushed a commit
that referenced
this pull request
May 12, 2022
* Switch server dependency from Express to Fastify * develop rebase and fix server address port * update bootstrap unit test with fastify and fix server address port in missing integration EA * add fastify type and test example * update EA tests with fastify server * remove fastify comment from @chainlink/types * fix test problem with agoric EA * update helpers server with fastify * remove express dependencies * remove unused code * update missing file * add changeset * remove http import in each EA * rename server variable name * remove hardcoded host * replace with arrow function * remove missing http imports * code refactor * merge .pnp.cjs * udpate with fastify blocksize-capital test, remove unused nock import * update ens integration test with fastify * fix problem with bootstrap tests, update status code and adding deleted customErrorMessage * fix problem with server log * change changeset and server log message * merge develop Co-authored-by: bogos <andre.neyra@gmail.com>
mjk90
added a commit
that referenced
this pull request
May 17, 2022
* Added GSR websocket adapter to sources * Added GSR websocket adapter integration tests * Added GSR README, updated env.json * Added changeset * Switch server dependency from Express to Fastify (#1638) * Switch server dependency from Express to Fastify * develop rebase and fix server address port * update bootstrap unit test with fastify and fix server address port in missing integration EA * add fastify type and test example * update EA tests with fastify server * remove fastify comment from @chainlink/types * fix test problem with agoric EA * update helpers server with fastify * remove express dependencies * remove unused code * update missing file * add changeset * remove http import in each EA * rename server variable name * remove hardcoded host * replace with arrow function * remove missing http imports * code refactor * merge .pnp.cjs * udpate with fastify blocksize-capital test, remove unused nock import * update ens integration test with fastify * fix problem with bootstrap tests, update status code and adding deleted customErrorMessage * fix problem with server log * change changeset and server log message * merge develop Co-authored-by: bogos <andre.neyra@gmail.com> * Updated gsr integration test to use fastify * Updated gsr token error handling Co-authored-by: Justin Kaseman <justinkaseman@live.com> Co-authored-by: bogos <andre.neyra@gmail.com>
alejoberardino
added a commit
that referenced
this pull request
May 17, 2022
* Release/v1.23.0 (#1858) * (feat): Add ENS adapter (#1841) * (feat): Add ENS adapter * Allow framework WARMUP_ENABLED override * (test): ENS adapter tests * Add changesets * Refresh yarn.lock * chore(deps): update dependency golang to v1.17.10 (#1853) Co-authored-by: Renovate Bot <bot@renovateapp.com> * chore(deps): update dependency @types/supertest to v2.0.12 (#1852) Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: karen-stepanyan <91897037+karen-stepanyan@users.noreply.github.com> * Switch server dependency from Express to Fastify (#1638) * Switch server dependency from Express to Fastify * develop rebase and fix server address port * update bootstrap unit test with fastify and fix server address port in missing integration EA * add fastify type and test example * update EA tests with fastify server * remove fastify comment from @chainlink/types * fix test problem with agoric EA * update helpers server with fastify * remove express dependencies * remove unused code * update missing file * add changeset * remove http import in each EA * rename server variable name * remove hardcoded host * replace with arrow function * remove missing http imports * code refactor * merge .pnp.cjs * udpate with fastify blocksize-capital test, remove unused nock import * update ens integration test with fastify * fix problem with bootstrap tests, update status code and adding deleted customErrorMessage * fix problem with server log * change changeset and server log message * merge develop Co-authored-by: bogos <andre.neyra@gmail.com> * Increased sleep period in finage integration test to ensure cache is populated (#1863) * Consume changesets (#1860) * Consume changesets * restarting workflow Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Morgan Kuphal <morgan.kuphal@smartcontract.com> * chore(deps): update dependency hardhat to v2.9.5 * fix(deps): update dependency airtable to v0.11.4 (#1867) Co-authored-by: Renovate Bot <bot@renovateapp.com> * fix(deps): update dependency ethers to v5.5.4 (#1868) Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: karen-stepanyan <91897037+karen-stepanyan@users.noreply.github.com> * chore(deps): update dependency @types/node to v14.18.18 (#1865) Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: karen-stepanyan <91897037+karen-stepanyan@users.noreply.github.com> * chore(deps): update dependency @types/node to v16.11.35 (#1869) Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: karen-stepanyan <91897037+karen-stepanyan@users.noreply.github.com> * Fix default EA host, add EA_HOST env var (#1871) * Consume changesets * Fix batch cache key generation by ignoring overrides & other excl props (#1875) * Consume changesets (#1873) * Consume changesets * Empty commit Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Austin Born <austinborn212@gmail.com> * Release v1.24.0 Co-authored-by: Morgan Kuphal <87319522+KuphJr@users.noreply.github.com> Co-authored-by: Justin Kaseman <justinkaseman@live.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: karen-stepanyan <91897037+karen-stepanyan@users.noreply.github.com> Co-authored-by: bogos <andre.neyra@gmail.com> Co-authored-by: Matthew Kelly <mkelly2@live.co.uk> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Morgan Kuphal <morgan.kuphal@smartcontract.com> Co-authored-by: Austin Born <austinborn212@gmail.com>
cds95
added a commit
that referenced
this pull request
May 23, 2022
* Release v1.22.0 * Release v1.23.0 * Release/v1.24.0 (#1877) * Release/v1.23.0 (#1858) * (feat): Add ENS adapter (#1841) * (feat): Add ENS adapter * Allow framework WARMUP_ENABLED override * (test): ENS adapter tests * Add changesets * Refresh yarn.lock * chore(deps): update dependency golang to v1.17.10 (#1853) Co-authored-by: Renovate Bot <bot@renovateapp.com> * chore(deps): update dependency @types/supertest to v2.0.12 (#1852) Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: karen-stepanyan <91897037+karen-stepanyan@users.noreply.github.com> * Switch server dependency from Express to Fastify (#1638) * Switch server dependency from Express to Fastify * develop rebase and fix server address port * update bootstrap unit test with fastify and fix server address port in missing integration EA * add fastify type and test example * update EA tests with fastify server * remove fastify comment from @chainlink/types * fix test problem with agoric EA * update helpers server with fastify * remove express dependencies * remove unused code * update missing file * add changeset * remove http import in each EA * rename server variable name * remove hardcoded host * replace with arrow function * remove missing http imports * code refactor * merge .pnp.cjs * udpate with fastify blocksize-capital test, remove unused nock import * update ens integration test with fastify * fix problem with bootstrap tests, update status code and adding deleted customErrorMessage * fix problem with server log * change changeset and server log message * merge develop Co-authored-by: bogos <andre.neyra@gmail.com> * Increased sleep period in finage integration test to ensure cache is populated (#1863) * Consume changesets (#1860) * Consume changesets * restarting workflow Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Morgan Kuphal <morgan.kuphal@smartcontract.com> * chore(deps): update dependency hardhat to v2.9.5 * fix(deps): update dependency airtable to v0.11.4 (#1867) Co-authored-by: Renovate Bot <bot@renovateapp.com> * fix(deps): update dependency ethers to v5.5.4 (#1868) Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: karen-stepanyan <91897037+karen-stepanyan@users.noreply.github.com> * chore(deps): update dependency @types/node to v14.18.18 (#1865) Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: karen-stepanyan <91897037+karen-stepanyan@users.noreply.github.com> * chore(deps): update dependency @types/node to v16.11.35 (#1869) Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: karen-stepanyan <91897037+karen-stepanyan@users.noreply.github.com> * Fix default EA host, add EA_HOST env var (#1871) * Consume changesets * Fix batch cache key generation by ignoring overrides & other excl props (#1875) * Consume changesets (#1873) * Consume changesets * Empty commit Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Austin Born <austinborn212@gmail.com> * Release v1.24.0 Co-authored-by: Morgan Kuphal <87319522+KuphJr@users.noreply.github.com> Co-authored-by: Justin Kaseman <justinkaseman@live.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: karen-stepanyan <91897037+karen-stepanyan@users.noreply.github.com> Co-authored-by: bogos <andre.neyra@gmail.com> Co-authored-by: Matthew Kelly <mkelly2@live.co.uk> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Morgan Kuphal <morgan.kuphal@smartcontract.com> Co-authored-by: Austin Born <austinborn212@gmail.com> * 1.25.0 * update master Co-authored-by: Jonas Hals <jonas@smartcontract.com> Co-authored-by: Austin Born <austinborn212@gmail.com> Co-authored-by: Morgan Kuphal <morgan.kuphal@smartcontract.com> Co-authored-by: Morgan Kuphal <87319522+KuphJr@users.noreply.github.com> Co-authored-by: Alejo Berardino <alejoberardino@gmail.com> Co-authored-by: Justin Kaseman <justinkaseman@live.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: karen-stepanyan <91897037+karen-stepanyan@users.noreply.github.com> Co-authored-by: bogos <andre.neyra@gmail.com> Co-authored-by: Matthew Kelly <mkelly2@live.co.uk> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Spike on profiling a more performant HTTP server dependency.
Changes
Steps to Test
POST/GET/healthGET/smokeQuality Assurance
yarn changesetif adapter source code was changed<ADAPTER_PACKAGE>/schemas/env.jsonand<ADAPTER_PACKAGE>/README.mdinfra-k8sconfiguration file.feature/x,chore/x,release/x,hotfix/x,fix/x) or is created from Clubhouse/Shortcut