Skip to content

Comments

test: bump d2-api version#2

Open
nshandra wants to merge 6 commits intomasterfrom
test/branch2
Open

test: bump d2-api version#2
nshandra wants to merge 6 commits intomasterfrom
test/branch2

Conversation

@nshandra
Copy link
Collaborator

📌 References

  • Issue: Closes #?

📝 Implementation

📹 Screenshots/Screen capture

🔥 Notes to the tester


getPassword() {
const suffix = Math.random();
const password = "myPassword" + suffix;

Check failure

Code scanning / CodeQL

Insecure randomness

This uses a cryptographically insecure random number generated at [Math.random()](1) in a security context.
getPassword2() {
const suffix = Math.random();
const suffix2 = Math.random();
const password = "myPassword" + suffix + suffix2;

Check failure

Code scanning / CodeQL

Insecure randomness

This uses a cryptographically insecure random number generated at [Math.random()](1) in a security context. This uses a cryptographically insecure random number generated at [Math.random()](2) in a security context.
})
).map(d2User => {
const res = this.buildUser(d2User);
const password = "myPassword" + Math.random();

Check failure

Code scanning / CodeQL

Insecure randomness

This uses a cryptographically insecure random number generated at [Math.random()](1) in a security context.
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.

1 participant