safe-global / safe-client-gateway / 25489915661

88%

push

github

migrating addressBook createdBy updatedBy to be userId (#3056)

* feat: allow gas-relay sponsorship for SafeWebAuthnSignerFactory.createSigner

Accepts createSigner calls to the official SafeWebAuthnSignerFactory v0.2.1
in the relay limit mapper, deriving a per-passkey rate-limit key from
keccak256(abi.encode(x, y, verifiers)) so passkey deployments don't share
a single global daily quota keyed on the factory address.

* refactor: address PR feedback on createSigner relay branch

- Throw UnofficialSignerFactoryError when isCreateSigner matches but the
  target isn't an official factory, mirroring UnofficialProxyFactoryError.
  Lets operators distinguish unofficial-factory attempts from unrecognised
  calldata.
- Assert the upstream ABI shape at module load in getSignerFactoryAbi() so
  a future @safe-global/safe-modules-deployments release that drops or
  renames createSigner/getSigner fails fast instead of silently drifting
  through the type cast.
- Add a top-level test exercising createSigner on a relay-supported chain
  (BSC) where the factory isn't deployed.

* refactor: address PR feedback on test setup and encoder return type

- signer-factory encoder.builder now returns Hex instead of Address.
  encodeFunctionData returns Hex (`0x${string}`); the calldata is not
  address-shaped. IEncoder defaults to Hex and other encoders match.
- Restructure SafeWebAuthnSignerFactory tests to fail loudly: lifted out
  of the per-chain loop into a describe.each over an explicit list of
  chains where v0.2.1 must be deployed ('1', '11155111'). beforeAll
  asserts both relay support and factory deployment, so config drift
  surfaces as a test failure instead of a silent skip.

* migrating createdBy updatedBy to be userId

* addressing review comments

updated migration script

resolver merge conflicts

map AB creator to their identity

add unknonw user if no email

address comments

remove fk constraint

---------

Co-authored-by: Daniel Dimitrov <daniel.d@safe.global>

4203 of 5063 branches covered (83.01%)

Branch coverage included in aggregate %.

15980 of 17500 relevant lines covered (91.31%)

569.49 hits per line