Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Refactor sudt with registry address #116

Merged
merged 15 commits into from
Apr 18, 2022
Merged

Refactor sudt with registry address #116

merged 15 commits into from
Apr 18, 2022

Conversation

jjyr
Copy link
Collaborator

@jjyr jjyr commented Apr 6, 2022 •
edited by Flouse
Loading

CHANGES

Core data structure

Address registry

Files

  • gw_registry_addr.h, address data structure
  • gw_eth_addr_reg.h
  • gw_eth_addr_reg.c
  • crates/state-validator

Changes

  • New builtin script: Ethereum address registry.
  • User deposit will mapping ethereum address to the Ethereum address registry.

Godwoken Syscalls

Files

  • sudt_utils.h
  • generator_utils.h
  • validator_utils.h

Changes

The following new syscall is added:

  • sys_get_registry_address_by_script_hash
  • sys_get_script_hash_by_registry_address

The following syscall is removed:

  • sys_get_script_hash_by_prefix - used to get script_hash by short_script_hash

Short script hash in sudt_utils.h is replaced by registry address

Signing

Files

  • crates/challenge-lock

Changes

  • Support signing tx and withdrawal in Ethereum tx format and EIP-712.
  • Using RollupConfig#chain_id in tx, withdrawal signature.

@jjyr jjyr requested a review from zeroqn as a code owner April 6, 2022 14:42
Flouse added a commit to Flouse/godwoken-docker-prebuilds that referenced this pull request Apr 7, 2022
@Flouse
GODWOKEN_REF := refs/pull/649/head & GODWOKEN_SCRIPTS_REF := refs/pul…
192cb34 
…l/116/head

- godwokenrises/godwoken#649
- godwokenrises/godwoken-scripts#116
Flouse
Flouse reviewed Apr 9, 2022
View reviewed changes
c/common.h Show resolved Hide resolved
c/common.h Outdated Show resolved Hide resolved
Flouse
Flouse reviewed Apr 10, 2022
View reviewed changes
c/generator_utils.h Show resolved Hide resolved
tests/src/testing_tool/chain.rs Outdated Show resolved Hide resolved
tests/src/script_tests/l2_scripts/examples.rs Outdated Show resolved Hide resolved
@Flouse Flouse mentioned this pull request Apr 12, 2022
Merged
@jjyr jjyr merged commit a23886b into godwokenrises:master Apr 18, 2022
Flouse added a commit to godwokenrises/godwoken-polyjuice that referenced this pull request Apr 18, 2022
@Flouse
Update the refs of Godwoken-scripts and Godwoken
b10ca26 
- godwokenrises/godwoken-scripts#116
- https://github.com/nervosnetwork/godwoken/commits/d4cc2a
@jjyr jjyr mentioned this pull request Apr 18, 2022
Merged
Flouse added a commit to godwokenrises/godwoken-polyjuice that referenced this pull request Apr 18, 2022
@Flouse
Update the refs of Godwoken-scripts and Godwoken
5905cb4 
- godwokenrises/godwoken-scripts#116
- https://github.com/nervosnetwork/godwoken/commits/d4cc2a
@Flouse Flouse mentioned this pull request Aug 12, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@zeroqn zeroqn zeroqn approved these changes

@Flouse Flouse Flouse approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jjyr @Flouse @zeroqn @magicalne