fix: enable LedgerStateFix for delegation (#5427)

This commit is contained in:
yinyiqian1
2025-05-10 10:36:11 -04:00
committed by GitHub
parent 3e152fec74
commit 28f50cb7cf
2 changed files with 6 additions and 4 deletions

View File

@@ -396,7 +396,7 @@ TRANSACTION(ttORACLE_DELETE, 52, OracleDelete, Delegation::delegatable, ({
}))
/** This transaction type fixes a problem in the ledger state */
TRANSACTION(ttLEDGER_STATE_FIX, 53, LedgerStateFix, Delegation::notDelegatable, ({
TRANSACTION(ttLEDGER_STATE_FIX, 53, LedgerStateFix, Delegation::delegatable, ({
{sfLedgerFixType, soeREQUIRED},
{sfOwner, soeOPTIONAL},
}))

View File

@@ -215,9 +215,7 @@ class Delegate_test : public beast::unit_test::suite
ter(terNO_ACCOUNT));
}
// for security reasons, AccountSet, SetRegularKey, SignerListSet,
// AccountDelete, DelegateSet are prohibited to be delegated to
// other accounts.
// non-delegatable transaction
{
env(delegate::set(gw, alice, {"SetRegularKey"}),
ter(tecNO_PERMISSION));
@@ -229,6 +227,10 @@ class Delegate_test : public beast::unit_test::suite
ter(tecNO_PERMISSION));
env(delegate::set(gw, alice, {"SetRegularKey"}),
ter(tecNO_PERMISSION));
env(delegate::set(gw, alice, {"EnableAmendment"}),
ter(tecNO_PERMISSION));
env(delegate::set(gw, alice, {"UNLModify"}), ter(tecNO_PERMISSION));
env(delegate::set(gw, alice, {"SetFee"}), ter(tecNO_PERMISSION));
}
}