Skip to content

acl connect cross-node integration test#115

Open
ldt1996 wants to merge 2 commits into
mainfrom
acl-connect-test
Open

acl connect cross-node integration test#115
ldt1996 wants to merge 2 commits into
mainfrom
acl-connect-test

Conversation

@ldt1996
Copy link
Copy Markdown

@ldt1996 ldt1996 commented May 4, 2026

No description provided.

@ldt1996 ldt1996 requested a review from a team as a code owner May 4, 2026 15:12
@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​harperdb/​acl-connect@​1.0.10671008192100

View full report

@socket-security
Copy link
Copy Markdown

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
Publisher changed: npm @harperdb/acl-connect is now published by ethan_arrowood

Author: ethan_arrowood

From: integrationTests/cluster/fixture-acl-connect/package.jsonnpm/@harperdb/acl-connect@1.0.10

ℹ Read more on: This package | This alert | What is unstable ownership?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Try to reduce the number of authors you depend on to reduce the risk to malicious actors gaining access to your supply chain. Packages should remove inactive collaborators with publishing rights from packages on npm.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/@harperdb/acl-connect@1.0.10. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

Warn Medium
Low adoption: npm @harperdb/acl-connect

Location: Package overview

From: integrationTests/cluster/fixture-acl-connect/package.jsonnpm/@harperdb/acl-connect@1.0.10

ℹ Read more on: This package | This alert | What are unpopular packages?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Unpopular packages may have less maintenance and contain other problems.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/@harperdb/acl-connect@1.0.10. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

Copy link
Copy Markdown
Member

@kriszyp kriszyp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work, thank you!

if (retries++ > 20) throw new Error('Timed out waiting for cluster to connect');
await sleep(200 * retries);
}
await sleep(500);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this necessary? At this point we should be good to go (with deployment)?

Comment thread integrationTests/cluster/aclConnectCrossNode.test.mjs
Comment thread integrationTests/cluster/aclConnectCrossNode.test.mjs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants