fix(docs): replace Math.random() nonce with crypto.randomUUID() in Wagmi integration example

Open Beginner friendly
#1,390 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
72/100
Issue type
Documentation
Clarity
Mostly clear
Activity status
Quiet
Tech stack
javascript

Research direction

Start in the Wagmi integration setup guide and compare its nonce example with the authenticate-users guide, which already uses window.crypto.randomUUID(). Update the example so the nonce uses the secure approach, then verify the rendered documentation is consistent and no Math.random() nonce remains.

Written by the indexing model from the issue text.

Description

The Wagmi integration setup guide uses Math.random() to generate a SIWE nonce. Math.random() is not cryptographically secure and should not be used for security-sensitive values. The authenticate-users guide already uses the correct window.crypto.randomUUID() approach — this makes the Wagmi integration consistent and safe.

Dominant language
JavaScript
Stars
337
Forks
798
Avg merge
7h 24m
Merged PRs (30d)
51

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from base/docs

All issues in base/docs

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.