Merge pull request #400 from perun-network/feat-secondary-settle

Revert: Remove settle secondary (#351)

Author: iljabvh

MAINTAINERS.md

@@ -10,6 +10,7 @@
 | Ilja von Hoessle  | [@iljabvh](https://github.com/iljabvh) | iljabvh |
 | Jens Winkle       | [@DragonDev1906](https://github.com/DragonDev1906) | jens#4601 |
 | Minh Huy Tran      | [@NhoxxKienn](https://github.com/NhoxxKienn) | NhoxxKienn |
+| Sophia Koehler  | [@sophia1ch](https://githun.com/sophia1ch)     | sophia#3072 |
 
 ## Emeritus Maintainers
 

NOTICE

@@ -44,6 +44,7 @@ PolyCrypt GmbH
     Ilja von Hoessle <ilja@perun.network>
     Jens Winkle <jens@perun.network>
     Minh Huy Tran <huy@perun.network>
+    Sophia Koehler <sophia@perun.network>
 
 Robert Bosch GmbH
     Manoranjith <ponraj.manoranjitha@in.bosch.com>

channel/adjudicator.go

@@ -103,11 +103,17 @@ type (
 
 	// An AdjudicatorReq collects all necessary information to make calls to the
 	// adjudicator.
+	//
+	// If the Secondary flag is set to true, it is assumed that this is an
+	// on-chain request that is executed by the other channel participants as well
+	// and the Adjudicator backend may run an optimized on-chain transaction
+	// protocol, possibly saving unnecessary double sending of transactions.
 	AdjudicatorReq struct {
-		Params *Params
-		Acc    wallet.Account
-		Tx     Transaction
-		Idx    Index // Always the own index
+		Params    *Params
+		Acc       wallet.Account
+		Tx        Transaction
+		Idx       Index // Always the own index
+		Secondary bool  // Optimized secondary call protocol
 	}
 
 	// SignedState represents a signed channel state including parameters.

channel/machine.go

@@ -249,6 +249,9 @@ func (m *machine) CurrentTX() Transaction {
 // AdjudicatorReq returns the adjudicator request for the current channel
 // transaction (the current state together with all participants' signatures on
 // it).
+//
+// The Secondary flag is left as false. Set it manually after creating the
+// request if you want to use optimized sencondary adjudication logic.
 func (m *machine) AdjudicatorReq() AdjudicatorReq {
 	return AdjudicatorReq{
 		Params: &m.params,
@@ -335,8 +338,9 @@ func (m *machine) EnableFinal() error {
 }
 
 // enableStaged checks that
-//   1. the current phase is `expected.From` and
-//   2. all signatures of the staging transactions have been set.
+//  1. the current phase is `expected.From` and
+//  2. all signatures of the staging transactions have been set.
+//
 // If successful, the staging transaction is promoted to be the current
 // transaction. If not, an error is returned.
 func (m *machine) enableStaged(expected PhaseTransition) error {

client/adjudicate.go

@@ -242,7 +242,7 @@ func (c *Channel) ForceUpdate(ctx context.Context, updater func(*channel.State))
 // to be mined.
 // Returns ChainNotReachableError if the connection to the blockchain network
 // fails when sending a transaction to / reading from the blockchain.
-func (c *Channel) Settle(ctx context.Context) (err error) {
+func (c *Channel) Settle(ctx context.Context, secondary bool) (err error) {
 	if !c.State().IsFinal {
 		err := c.ensureRegistered(ctx)
 		if err != nil {
@@ -268,7 +268,7 @@ func (c *Channel) Settle(ctx context.Context) (err error) {
 	}
 
 	// Withdraw.
-	err = c.withdraw(ctx)
+	err = c.withdraw(ctx, secondary)
 	if err != nil {
 		return
 	}
@@ -303,14 +303,15 @@ func (c *Channel) Settle(ctx context.Context) (err error) {
 	return nil
 }
 
-func (c *Channel) withdraw(ctx context.Context) error {
+func (c *Channel) withdraw(ctx context.Context, secondary bool) error {
 	switch {
 	case c.IsLedgerChannel():
 		subStates, err := c.subChannelStateMap()
 		if err != nil {
 			return errors.WithMessage(err, "creating sub-channel state map")
 		}
 		req := c.machine.AdjudicatorReq()
+		req.Secondary = secondary
 		if err := c.adjudicator.Withdraw(ctx, req, subStates); err != nil {
 			return errors.WithMessage(err, "calling Withdraw")
 		}

client/test/bob.go

@@ -57,7 +57,7 @@ func (r *Bob) exec(_cfg ExecConfig, ch *paymentChannel, propHandler *acceptNextP
 	ch.sendFinal()
 
 	// 4th Settle channel
-	ch.settle()
+	ch.settleSecondary()
 
 	// 4th final stage
 	r.waitStage()

client/test/channel.go

@@ -168,10 +168,18 @@ func (ch *paymentChannel) recvFinal() {
 }
 
 func (ch *paymentChannel) settle() {
+	ch.settleImpl(false)
+}
+
+func (ch *paymentChannel) settleSecondary() {
+	ch.settleImpl(true)
+}
+
+func (ch *paymentChannel) settleImpl(secondary bool) {
 	ctx, cancel := context.WithTimeout(context.Background(), ch.r.timeout)
 	defer cancel()
 
-	ch.r.RequireNoError(ch.Settle(ctx))
+	ch.r.RequireNoError(ch.Settle(ctx, secondary))
 	ch.assertBals(ch.State())
 
 	if ch.IsSubChannel() {

client/test/fund.go

@@ -129,7 +129,7 @@ func runFredFridaTest(
 	require.IsType(t, &client.ChannelFundingError{}, err)
 	require.NotNil(t, chFrida)
 	// Frida settles the channel.
-	require.NoError(t, chFrida.Settle(ctx))
+	require.NoError(t, chFrida.Settle(ctx, false))
 
 	// Fred gets the channel and settles it afterwards.
 	chFred := <-chsFred
@@ -141,7 +141,7 @@ func runFredFridaTest(
 		require.NoError(t, ctx.Err())
 	}
 	// Fred settles the channel.
-	require.NoError(t, chFred.Settle(ctx))
+	require.NoError(t, chFred.Settle(ctx, false))
 
 	// Test the final balances.
 	balancesAfter := channel.Balances{

client/test/multiledger_dispute.go

@@ -31,6 +31,7 @@ import (
 // TestMultiLedgerDispute runs an end-to-end test of the multi-ledger
 // functionality in the dispute case for the implementation specified in the
 // test setup.
+//
 //nolint:revive // test.Test... stutters but this is OK in this special case.
 func TestMultiLedgerDispute(
 	ctx context.Context,
@@ -127,9 +128,9 @@ func TestMultiLedgerDispute(
 	require.NoError(err)
 
 	// Settle.
-	err = chAliceBob.Settle(ctx)
+	err = chAliceBob.Settle(ctx, false)
 	require.NoError(err)
-	err = chBobAlice.Settle(ctx)
+	err = chBobAlice.Settle(ctx, false)
 	require.NoError(err)
 
 	// Close the channels.

client/test/multiledger_happy.go

@@ -27,6 +27,7 @@ import (
 // TestMultiLedgerHappy runs an end-to-end test of the multi-ledger
 // functionality in the optimistic case for the implementation specified in the
 // test setup.
+//
 //nolint:revive // test.Test... stutters but this is OK in this special case.
 func TestMultiLedgerHappy(ctx context.Context, t *testing.T, mlt MultiLedgerSetup, challengeDuration uint64) {
 	require := require.New(t)
@@ -90,8 +91,8 @@ func TestMultiLedgerHappy(ctx context.Context, t *testing.T, mlt MultiLedgerSetu
 	require.NoError(err)
 
 	// Close channel.
-	err = chAliceBob.Settle(ctx)
+	err = chAliceBob.Settle(ctx, false)
 	require.NoError(err)
-	err = chBobAlice.Settle(ctx)
+	err = chBobAlice.Settle(ctx, false)
 	require.NoError(err)
 }