New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat multiple backends #410

Open

sophia1ch wants to merge 34 commits into hyperledger-labs:main from perun-network:feat-mult-backends

Contributor

sophia1ch commented Dec 11, 2024

Description

This PR introduces the functionality of using multiple backends that implement the go-perun backend. This functionality includes:

using maps of addresses for each user (one address per backend)
using maps of channel IDs
backendID to differentiate the backends being used
adjustments in the state and params

These changes allow for EVM to non-EVM payment-channels.

sophia1ch added 14 commits

August 21, 2024 14:09


          feat(wallet, channel): Change backend to map of backends

66fee75

feat(Asset, Adjudicator): Add AssetID struct to identify assets with LedgerID and BackendID
feat(persistence, wire): Use map[int]Address to identify clients

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix(multi): Add AssetIDKey struct to fix the map lookups for funders …

28685ea

…and adjudicators.

fix(randomizer): Lookup backend opts to correctly initialize address maps.
fix(test): Generalize tests to be used in other backends.

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix(all): Use map[int]ID instead of id for each backend.

ef11de9

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix(all): Add map of IDs

ab8e60b

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix(channel/backend): Revert change in verify.

13eeddf

fix(wallet/wallet, wallet/account): Revert changed definition.
feat(client): Add wallet map to client for each backend.

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix(Adjudicator): Change to singular channel ID in events.

8181cb0

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix(client/test): Generalize role test.

794298a

Signed-off-by: Sophia Koehler <sophia@perun.network>


          feat(channel): Add backend field check.

969c6a7

feat(client/test, wire): Add backendID arguments.

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix(subchannel_dispute): Generalize event watcher.

10c9b28

Signed-off-by: Sophia Koehler <sophia@perun.network>


          feat: Asset Address function to handle encoding in backends.

3c1e261

Signed-off-by: Sophia Koehler <sophia@perun.network>


          feat(apps/payment, backend/sim, channel/test, client/test, log, walle…

a695d14

…t, ): Specify backend for randomizers

feat(channel/multi): Change AssetID to interface
feat(channel/persistence, client/test): Generalize test for backends

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix(asset.go, allocation.go): Adjust Address() to return byte[] inste…

8589eb8

…ad of string.

fix(params.go): Change IDKey to encode map entries sorted by keys.
fix(backend.go): Use BackendID in argument to sign depending on backend.
fix(address.go): Sort Addresses by keys before adding them to Keys.

Signed-off-by: Sophia Koehler <sophia@perun.network>


          chore: Remove unnecessary logs

39095c1

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix: Remove repeated assetIDs in LedgerID array

98e11ea

Signed-off-by: Sophia Koehler <sophia@perun.network>

sophia1ch mentioned this pull request

Adjust to stellar hyperledger-labs/perun-eth-backend#54

Open

sophia1ch marked this pull request as ready for review

December 18, 2024 07:18

sophia1ch marked this pull request as draft

December 18, 2024 07:36

sophia1ch marked this pull request as ready for review

December 18, 2024 09:21

sophia1ch added 2 commits

December 18, 2024 10:27


          chore: update go.sum, gofmt some files

46de2d9

Signed-off-by: Sophia Koehler <sophia@perun.network>


          Merge branch 'main' into feat-mult-backends

dac83b9

Signed-off-by: Sophia Koehler <sophia@perun.network>

sophia1ch force-pushed the feat-mult-backends branch from 408a062 to dac83b9 Compare

December 19, 2024 09:23

sophia1ch added 9 commits

December 19, 2024 10:32


          fix(wallet/backend.go): Return just one error instead of concatenatio…

5e5bfb3

…n of errors

chore(All): Run gofmt and sort imports

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix(golangci.yml): Exclude generated file from linter

70036df

chore: Refactor for linter

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix(ci): Use go1.18 for the any type in proto

19a792f

fix(golangci): Fix structure

Signed-off-by: Sophia Koehler <sophia@perun.network>


          fix(ci): Use glinter 1.45 which supports go1.18

fd8bf8a

Signed-off-by: Sophia Koehler <sophia@perun.network>


          chore(all): Add nolint flags for conversions

98b900b

Signed-off-by: Sophia Koehler <sophia@perun.network>


          chore(all): remove unnecessary flags

2f24516

Signed-off-by: Sophia Koehler <sophia@perun.network>


          chore(all): remove unnecessary flags

8622a49

Signed-off-by: Sophia Koehler <sophia@perun.network>


          chore(all): remove unnecessary flags

caa8d4c

Signed-off-by: Sophia Koehler <sophia@perun.network>


          chore(all): remove unnecessary flags

b272564

Signed-off-by: Sophia Koehler <sophia@perun.network>

sophia1ch added 5 commits

December 19, 2024 13:47


          chore(all): remove unnecessary flags

2c079f1

Signed-off-by: Sophia Koehler <sophia@perun.network>


          chore(all): remove unnecessary flags

ced8fb5

Signed-off-by: Sophia Koehler <sophia@perun.network>


          chore(all): remove unnecessary flags

056e726

Signed-off-by: Sophia Koehler <sophia@perun.network>


          chore(all): Rename and add comments to exported functions

d73ca03

Signed-off-by: Sophia Koehler <sophia@perun.network>


          chore: Rename variables

2d98975

Signed-off-by: Sophia Koehler <sophia@perun.network>

Contributor Author

sophia1ch commented Dec 23, 2024

Description

This PR introduces the functionality of using multiple backends that implement the go-perun backend. This functionality includes:

using maps of addresses for each user (one address per backend)

using maps of channel IDs

backendID to differentiate the backends being used

adjustments in the state and params

These changes allow for EVM to non-EVM payment-channels.

Additionally, I updated the CI configuration to use Go 1.18 due to compatibility issues with the auto-generated proto file. Updating directly to Go 1.22 introduced several additional errors, primarily related to the linter. A separate issue has been created to address the Go 1.22 update.


          fix(params, address): Fix index allocation

d463e82

Signed-off-by: Sophia Koehler <sophia@perun.network>

iljabvh self-requested a review

January 7, 2025 15:59

This was referenced Jan 14, 2025

Remove channelID map #413

Open

Fix egoistic funding #414

Draft

sophia1ch added 3 commits

January 20, 2025 08:03


          refactor(/apps/payment, /sim/wallet, /channel, /client, /wallet, /wir…

7169c29

…e): Name test backend ID in tests

refactor(/multi, /client/test): Rename AssetID to MultiLedgerID

Signed-off-by: Sophia Koehler <sophia@perun.network>


          refactor: Name TestBackendID, rename multiLedgerID to LedgerBackendID

c1fe0e2

Signed-off-by: Sophia Koehler <sophia@perun.network>


          refactor: Fix comment on exported function

a6d4a4a

Signed-off-by: Sophia Koehler <sophia@perun.network>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet