Skip to content

nuxt-bridge ft nitro - #3450

Closed
wommy wants to merge 3 commits into
advplyr:masterfrom
wommy:nuxtBridge-n-nitro
Closed

nuxt-bridge ft nitro#3450
wommy wants to merge 3 commits into
advplyr:masterfrom
wommy:nuxtBridge-n-nitro

Conversation

@wommy

@wommy wommy commented Sep 24, 2024

Copy link
Copy Markdown
Contributor

No description provided.

Comment on lines +27 to +31
var uuid = "xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx".replace(/[xy]/g, function(c) {
var r = (d + Math.random()*16)%16 | 0;
d = Math.floor(d/16);
return (c=="x" ? r : (r&0x7|0x8)).toString(16);
});

Check failure

Code scanning / CodeQL

Insecure randomness

This uses a cryptographically insecure random number generated at [Math.random()](1) in a security context.

@wommy wommy Sep 25, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this is in the carried over code; not mine

i could PR this too

@wommy

wommy commented Sep 25, 2024

Copy link
Copy Markdown
Contributor Author

futurepress/epub.js#1399

@wommy

wommy commented Oct 7, 2024

Copy link
Copy Markdown
Contributor Author

#3426

discussion here

@wommy
wommy force-pushed the nuxtBridge-n-nitro branch from 87d105f to c840d37 Compare October 7, 2024 14:54
@wommy

wommy commented Oct 7, 2024

Copy link
Copy Markdown
Contributor Author

nuxt/bridge#1078

issue w/ hookable, defu version mismatch

@wommy wommy mentioned this pull request Oct 16, 2024
@wommy
wommy force-pushed the nuxtBridge-n-nitro branch from c840d37 to b54882b Compare October 17, 2024 01:25
@wommy
wommy force-pushed the nuxtBridge-n-nitro branch from b54882b to e77609e Compare October 17, 2024 02:09
@advplyr advplyr closed this Feb 1, 2026
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.

3 participants