Skip to content

feat: add device ID hashing package COMPASS-8443 #532

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.

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

Merged
merged 6 commits into from
May 5, 2025
Merged

Conversation

gagik
Copy link
Collaborator

@gagik gagik commented May 2, 2025

Creates a consistent, implementation-agnostic hash from a given raw machine ID. The machine ID should originate from node-machine-id or native-machine-id depending on the platform. The hash is generated using SHA-256 and is designed to be consistent with the Atlas CLI.

} else {
resolve('unknown');
}
}, timeout);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
}, timeout);
}, timeout).unref?.();

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

TIL

@gagik gagik force-pushed the gagik/add-hashing-lib branch 2 times, most recently from c344e1c to db019bb Compare May 5, 2025 09:29
@gagik gagik force-pushed the gagik/add-hashing-lib branch from db019bb to 115f688 Compare May 5, 2025 09:29
@gagik
Copy link
Collaborator Author

gagik commented May 5, 2025

There seems to be some unrelated SBOM dependency failing issue; going ahead with merge.

@gagik gagik merged commit 0e0ff3e into main May 5, 2025
3 of 6 checks passed
@gagik gagik deleted the gagik/add-hashing-lib branch May 5, 2025 09:31
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.

2 participants