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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

build(deps): [Snyk] Upgrade @hashgraph/sdk from 2.50.0 to 2.51.0 #3069

Merged
merged 3 commits into from
Oct 9, 2024
Merged

build(deps): [Snyk] Upgrade @hashgraph/sdk from 2.50.0 to 2.51.0 #3069

merged 3 commits into from
Oct 9, 2024

Conversation

swirlds-automation
Copy link
Contributor

snyk-top-banner

Snyk has created this PR to upgrade @hashgraph/sdk from 2.50.0 to 2.51.0.

ℹ️ Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

  • The recommended version is 1 version ahead of your current version.

  • The recommended version was released on 22 days ago.

Release notes
Package name: @hashgraph/sdk from @hashgraph/sdk GitHub release notes

Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.

For more information:

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 4, 2024 •
edited
Loading

Acceptance Tests

  18 files  246 suites   32m 13s ⏱️
602 tests 596 ✔️ 4 💤 2 ❌
694 runs  686 ✔️ 4 💤 4 ❌

Results for commit a28dff9.

♻️ This comment has been updated with latest results.

@ebadiere ebadiere changed the title [Snyk] Upgrade @hashgraph/sdk from 2.50.0 to 2.51.0 build(deps)[Snyk] Upgrade @hashgraph/sdk from 2.50.0 to 2.51.0 Oct 4, 2024
@snyk-bot @ebadiere
fix: upgrade @hashgraph/sdk from 2.50.0 to 2.51.0
5149307 
Snyk has created this PR to upgrade @hashgraph/sdk from 2.50.0 to 2.51.0.

See this package in npm:
@hashgraph/sdk

See this project in Snyk:
https://app.snyk.io/org/json-rpc-relay/project/baf29319-5e17-4aff-8e1f-4108d3e10c45?utm_source=github&utm_medium=referral&page=upgrade-pr

Signed-off-by: ebadiere <[email protected]>
@ebadiere ebadiere force-pushed the snyk-upgrade-b012431a9cf06bc398c1c60d7aec1c51 branch from 0c0e11d to 5149307 Compare October 4, 2024 14:57
@ebadiere ebadiere changed the title build(deps)[Snyk] Upgrade @hashgraph/sdk from 2.50.0 to 2.51.0 build(deps): [Snyk] Upgrade @hashgraph/sdk from 2.50.0 to 2.51.0 Oct 4, 2024
@ebadiere ebadiere added the dependencies Pull requests that update a dependency file label Oct 4, 2024
@ebadiere ebadiere added this to the 0.58.0 milestone Oct 4, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 4, 2024 •
edited
Loading

Tests

       3 files     299 suites   19s ⏱️
1 354 tests 1 353 ✔️ 1 💤 0 ❌
1 363 runs  1 362 ✔️ 1 💤 0 ❌

Results for commit f4bdf00.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 8, 2024 •
edited
Loading

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled validates enforcement of request id. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 36,566.7 ms

Heap Statistics (before vs after executing the test):

  • Total Heap Size: increased with 1.72 MB
  • Total Heap Size Executable: no changes
  • Total Physical Size: decreased with 344.06 KB
  • Total Available Size: increased with 3.06 MB
  • Total Global Handles Size: no changes
  • Used Global Handles Size: decreased with 64.00 bytes
  • Used Heap Size: decreased with 3.28 MB
  • Heap Size Limit: no changes
  • Malloced Memory: decreased with 768.00 bytes
  • External Memory: no changes
  • Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

  • Old Space:

    • Space Size: increased with 2.10 MB
    • Space Used Size: increased with 2.08 MB
    • Space Available Size: increased with 146.45 KB
    • Physical Space Size: increased with 2.10 MB

  • Large Object Space:

    • Space Size: increased with 835.58 KB
    • Space Used Size: increased with 813.50 KB
    • Space Available Size: no changes
    • Physical Space Size: increased with 835.58 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Oct 8, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@quiet-node quiet-node merged commit 670e5fa into main Oct 9, 2024
42 of 43 checks passed
@quiet-node quiet-node deleted the snyk-upgrade-b012431a9cf06bc398c1c60d7aec1c51 branch October 9, 2024 01:48
@codecov Codecov
Copy link

codecov bot commented Oct 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.84%. Comparing base (0ba2162) to head (f4bdf00).
Report is 1 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #3069   +/-   ##
=======================================
  Coverage   84.84%   84.84%           
=======================================
  Files          59       59           
  Lines        3939     3939           
  Branches      788      788           
=======================================
  Hits         3342     3342           
  Misses        359      359           
  Partials      238      238
Flag Coverage Δ
relay 84.95% <ø> (ø)
server 83.43% <ø> (ø)
ws-server 97.87% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ebadiere ebadiere ebadiere approved these changes

@quiet-node quiet-node quiet-node approved these changes

@georgi-l95 georgi-l95 Awaiting requested review from georgi-l95 georgi-l95 is a code owner

@Ivo-Yankov Ivo-Yankov Awaiting requested review from Ivo-Yankov Ivo-Yankov is a code owner

Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
0.58.0
Development

Successfully merging this pull request may close these issues.
4 participants
@swirlds-automation @ebadiere @quiet-node @snyk-bot