Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Upgrade Artillery dependency #1910

Merged
merged 5 commits into from
Nov 2, 2023

Conversation

vardan10
Copy link
Contributor

@vardan10 vardan10 commented Nov 1, 2023

What was the problem?

This PR resolves #1901

How was it solved?

  • Fixed gateway error for RPC requests.
  • Used a custom engine as mentioned in socket.io docs to fix stress tests
  • Improve block indexing speed
  • Fix semgrep on CI

How was it tested?

Local

@vardan10 vardan10 linked an issue Nov 1, 2023 that may be closed by this pull request
@vardan10 vardan10 self-assigned this Nov 1, 2023
Copy link

codecov bot commented Nov 1, 2023

Codecov Report

Merging #1910 (2604694) into release/0.7.0 (8a5104d) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff               @@
##           release/0.7.0    #1910   +/-   ##
==============================================
  Coverage          66.62%   66.62%           
==============================================
  Files                298      298           
  Lines               5250     5250           
  Branches             949      949           
==============================================
  Hits                3498     3498           
  Misses              1752     1752           
Files Coverage Δ
services/blockchain-connector/config.js 100.00% <100.00%> (ø)
services/blockchain-indexer/config.js 100.00% <100.00%> (ø)

@vardan10 vardan10 marked this pull request as ready for review November 1, 2023 15:59
@sameersubudhi sameersubudhi force-pushed the 1901-upgrade-Artillery-dependency branch from 78e03e2 to 2604694 Compare November 2, 2023 11:31
@sameersubudhi sameersubudhi merged commit 5cef435 into release/0.7.0 Nov 2, 2023
7 of 8 checks passed
@sameersubudhi sameersubudhi deleted the 1901-upgrade-Artillery-dependency branch November 2, 2023 13:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade Artillery dependency
3 participants