WalletConnect / web3modal

A single Web3 provider solution for all Wallets

Home Page:https://web3modal.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[bug] SIWE Issued At is undefined

chris13524 opened this issue · comments

Link to minimal reproducible example

https://lab.web3modal.com/library/wagmi-all/

Summary

Login with email

image

List of related npm package versions

latest

I believe it's not a bug, and just the documentation is not covering all the getMessageParams possible arguments. Checking the source code, I can see that you can pass iat, which will work fine. E.g.:

	const getMessageParams = async () => ({
		domain: window.location.host,
		uri: window.location.origin,
		chains: [1],
		statement: 'Please sign with your account',
		iat: new Date().toISOString(),
	});

Tested with version 4.2.2

I believe it's not a bug

It is a bug. This field is required as per ERC-4361, so if it was a docs issue then not providing it should be a compile-time error. Second, even if it was optional, the field shouldn't be included at all instead of being an "undefined" string