@aztec/slasher 0.0.1-commit.9b94fc1 → 0.0.1-commit.9badcec54
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +51 -65
- package/dest/config.d.ts +1 -1
- package/dest/config.d.ts.map +1 -1
- package/dest/config.js +28 -28
- package/dest/factory/create_facade.d.ts +5 -4
- package/dest/factory/create_facade.d.ts.map +1 -1
- package/dest/factory/create_facade.js +26 -3
- package/dest/factory/create_implementation.d.ts +7 -8
- package/dest/factory/create_implementation.d.ts.map +1 -1
- package/dest/factory/create_implementation.js +8 -56
- package/dest/factory/get_settings.d.ts +4 -4
- package/dest/factory/get_settings.d.ts.map +1 -1
- package/dest/factory/get_settings.js +3 -3
- package/dest/factory/index.d.ts +2 -2
- package/dest/factory/index.d.ts.map +1 -1
- package/dest/factory/index.js +1 -1
- package/dest/generated/slasher-defaults.d.ts +19 -0
- package/dest/generated/slasher-defaults.d.ts.map +1 -0
- package/dest/generated/slasher-defaults.js +19 -0
- package/dest/index.d.ts +2 -3
- package/dest/index.d.ts.map +1 -1
- package/dest/index.js +1 -2
- package/dest/null_slasher_client.d.ts +3 -4
- package/dest/null_slasher_client.d.ts.map +1 -1
- package/dest/null_slasher_client.js +1 -4
- package/dest/slash_offenses_collector.d.ts +5 -8
- package/dest/slash_offenses_collector.d.ts.map +1 -1
- package/dest/slash_offenses_collector.js +9 -18
- package/dest/slasher_client.d.ts +112 -0
- package/dest/slasher_client.d.ts.map +1 -0
- package/dest/{tally_slasher_client.js → slasher_client.js} +30 -39
- package/dest/slasher_client_facade.d.ts +7 -9
- package/dest/slasher_client_facade.d.ts.map +1 -1
- package/dest/slasher_client_facade.js +6 -9
- package/dest/slasher_client_interface.d.ts +7 -21
- package/dest/slasher_client_interface.d.ts.map +1 -1
- package/dest/slasher_client_interface.js +1 -4
- package/dest/stores/offenses_store.d.ts +6 -12
- package/dest/stores/offenses_store.d.ts.map +1 -1
- package/dest/stores/offenses_store.js +8 -25
- package/dest/watchers/attestations_block_watcher.d.ts +7 -6
- package/dest/watchers/attestations_block_watcher.d.ts.map +1 -1
- package/dest/watchers/attestations_block_watcher.js +40 -34
- package/dest/watchers/epoch_prune_watcher.d.ts +9 -7
- package/dest/watchers/epoch_prune_watcher.d.ts.map +1 -1
- package/dest/watchers/epoch_prune_watcher.js +59 -15
- package/package.json +16 -14
- package/src/config.ts +31 -28
- package/src/factory/create_facade.ts +35 -6
- package/src/factory/create_implementation.ts +25 -106
- package/src/factory/get_settings.ts +8 -8
- package/src/factory/index.ts +1 -1
- package/src/generated/slasher-defaults.ts +21 -0
- package/src/index.ts +1 -2
- package/src/null_slasher_client.ts +2 -6
- package/src/slash_offenses_collector.ts +16 -20
- package/src/{tally_slasher_client.ts → slasher_client.ts} +37 -48
- package/src/slasher_client_facade.ts +7 -12
- package/src/slasher_client_interface.ts +6 -21
- package/src/stores/offenses_store.ts +11 -34
- package/src/watcher.ts +1 -1
- package/src/watchers/attestations_block_watcher.ts +57 -44
- package/src/watchers/epoch_prune_watcher.ts +87 -24
- package/dest/empire_slasher_client.d.ts +0 -190
- package/dest/empire_slasher_client.d.ts.map +0 -1
- package/dest/empire_slasher_client.js +0 -572
- package/dest/stores/payloads_store.d.ts +0 -29
- package/dest/stores/payloads_store.d.ts.map +0 -1
- package/dest/stores/payloads_store.js +0 -125
- package/dest/tally_slasher_client.d.ts +0 -125
- package/dest/tally_slasher_client.d.ts.map +0 -1
- package/src/empire_slasher_client.ts +0 -657
- package/src/stores/payloads_store.ts +0 -146
|
@@ -1,11 +1,10 @@
|
|
|
1
1
|
import { EthAddress } from '@aztec/aztec.js/addresses';
|
|
2
2
|
import type { EpochCache } from '@aztec/epoch-cache';
|
|
3
|
-
import { RollupContract, SlasherContract,
|
|
3
|
+
import { RollupContract, SlasherContract, SlashingProposerContract } from '@aztec/ethereum/contracts';
|
|
4
4
|
import { maxBigint } from '@aztec/foundation/bigint';
|
|
5
5
|
import { SlotNumber } from '@aztec/foundation/branded-types';
|
|
6
6
|
import { compactArray, partition, times } from '@aztec/foundation/collection';
|
|
7
7
|
import { createLogger } from '@aztec/foundation/log';
|
|
8
|
-
import { sleep } from '@aztec/foundation/sleep';
|
|
9
8
|
import type { DateProvider } from '@aztec/foundation/timer';
|
|
10
9
|
import type { Prettify } from '@aztec/foundation/types';
|
|
11
10
|
import type { SlasherConfig } from '@aztec/stdlib/interfaces/server';
|
|
@@ -14,7 +13,6 @@ import {
|
|
|
14
13
|
OffenseType,
|
|
15
14
|
type ProposerSlashAction,
|
|
16
15
|
type ProposerSlashActionProvider,
|
|
17
|
-
type SlashPayloadRound,
|
|
18
16
|
getEpochsForRound,
|
|
19
17
|
getSlashConsensusVotesFromOffenses,
|
|
20
18
|
} from '@aztec/stdlib/slashing';
|
|
@@ -31,8 +29,8 @@ import type { SlasherClientInterface } from './slasher_client_interface.js';
|
|
|
31
29
|
import type { SlasherOffensesStore } from './stores/offenses_store.js';
|
|
32
30
|
import type { Watcher } from './watcher.js';
|
|
33
31
|
|
|
34
|
-
/** Settings used in the
|
|
35
|
-
export type
|
|
32
|
+
/** Settings used in the slasher client, loaded from the L1 contracts during initialization */
|
|
33
|
+
export type SlasherSettings = Prettify<
|
|
36
34
|
SlashRoundMonitorSettings &
|
|
37
35
|
SlashOffensesCollectorSettings & {
|
|
38
36
|
slashingLifetimeInRounds: number;
|
|
@@ -46,11 +44,14 @@ export type TallySlasherSettings = Prettify<
|
|
|
46
44
|
}
|
|
47
45
|
>;
|
|
48
46
|
|
|
49
|
-
export type
|
|
50
|
-
Pick<
|
|
47
|
+
export type SlasherClientConfig = SlashOffensesCollectorConfig &
|
|
48
|
+
Pick<
|
|
49
|
+
SlasherConfig,
|
|
50
|
+
'slashValidatorsAlways' | 'slashValidatorsNever' | 'slashExecuteRoundsLookBack' | 'slashMaxPayloadSize'
|
|
51
|
+
>;
|
|
51
52
|
|
|
52
53
|
/**
|
|
53
|
-
* The
|
|
54
|
+
* The Slasher client is responsible for managing slashable offenses using
|
|
54
55
|
* the consensus-based slashing model where proposers vote on individual validator offenses.
|
|
55
56
|
*
|
|
56
57
|
* The client subscribes to several slash watchers that emit offenses and tracks them. When the slasher is the
|
|
@@ -74,22 +75,16 @@ export type TallySlasherClientConfig = SlashOffensesCollectorConfig &
|
|
|
74
75
|
* - Validators that reach the quorum threshold are slashed. A vote for slashing N units is also considered
|
|
75
76
|
* a vote for slashing N-1, N-2, ..., 1 units. The system slashes for the largest amount that reaches quorum.
|
|
76
77
|
* - The client monitors executable rounds and triggers execution when appropriate.
|
|
77
|
-
*
|
|
78
|
-
* Differences from Empire model
|
|
79
|
-
* - No fixed slash payloads - votes are for individual validator offenses encoded in bytes
|
|
80
|
-
* - The L1 contract determines which offenses reach quorum rather than nodes agreeing on a payload
|
|
81
|
-
* - Proposers vote directly on which validators to slash and by how much
|
|
82
|
-
* - Uses a slash offset to vote on validators from past rounds (e.g., round N votes on round N-2)
|
|
83
78
|
*/
|
|
84
|
-
export class
|
|
79
|
+
export class SlasherClient implements ProposerSlashActionProvider, SlasherClientInterface {
|
|
85
80
|
protected unwatchCallbacks: (() => void)[] = [];
|
|
86
81
|
protected roundMonitor: SlashRoundMonitor;
|
|
87
82
|
protected offensesCollector: SlashOffensesCollector;
|
|
88
83
|
|
|
89
84
|
constructor(
|
|
90
|
-
private config:
|
|
91
|
-
private settings:
|
|
92
|
-
private
|
|
85
|
+
private config: SlasherClientConfig,
|
|
86
|
+
private settings: SlasherSettings,
|
|
87
|
+
private slashingProposer: SlashingProposerContract,
|
|
93
88
|
private slasher: SlasherContract,
|
|
94
89
|
private rollup: RollupContract,
|
|
95
90
|
watchers: Watcher[],
|
|
@@ -103,14 +98,14 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
|
|
|
103
98
|
}
|
|
104
99
|
|
|
105
100
|
public async start() {
|
|
106
|
-
this.log.debug('Starting
|
|
101
|
+
this.log.debug('Starting slasher client...');
|
|
107
102
|
|
|
108
103
|
this.roundMonitor.start();
|
|
109
104
|
await this.offensesCollector.start();
|
|
110
105
|
|
|
111
106
|
// Listen for RoundExecuted events
|
|
112
107
|
this.unwatchCallbacks.push(
|
|
113
|
-
this.
|
|
108
|
+
this.slashingProposer.listenToRoundExecuted(
|
|
114
109
|
({ round, slashCount, l1BlockHash }) =>
|
|
115
110
|
void this.handleRoundExecuted(round, slashCount, l1BlockHash).catch(err =>
|
|
116
111
|
this.log.error('Error handling round executed', err),
|
|
@@ -121,15 +116,13 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
|
|
|
121
116
|
// Check for round changes
|
|
122
117
|
this.unwatchCallbacks.push(this.roundMonitor.listenToNewRound(round => this.handleNewRound(round)));
|
|
123
118
|
|
|
124
|
-
this.log.info(`Started
|
|
119
|
+
this.log.info(`Started slasher client`);
|
|
125
120
|
return Promise.resolve();
|
|
126
121
|
}
|
|
127
122
|
|
|
128
|
-
/**
|
|
129
|
-
* Stop the tally slasher client
|
|
130
|
-
*/
|
|
123
|
+
/** Stop the slasher client */
|
|
131
124
|
public async stop() {
|
|
132
|
-
this.log.debug('Stopping
|
|
125
|
+
this.log.debug('Stopping slasher client...');
|
|
133
126
|
|
|
134
127
|
for (const unwatchCallback of this.unwatchCallbacks) {
|
|
135
128
|
unwatchCallback();
|
|
@@ -138,9 +131,7 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
|
|
|
138
131
|
this.roundMonitor.stop();
|
|
139
132
|
await this.offensesCollector.stop();
|
|
140
133
|
|
|
141
|
-
|
|
142
|
-
await sleep(2000);
|
|
143
|
-
this.log.info('Tally Slasher client stopped');
|
|
134
|
+
this.log.info('Slasher client stopped');
|
|
144
135
|
}
|
|
145
136
|
|
|
146
137
|
/** Returns the current config */
|
|
@@ -155,11 +146,11 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
|
|
|
155
146
|
|
|
156
147
|
/** Triggered on a time basis when we enter a new slashing round. Clears expired offenses. */
|
|
157
148
|
protected async handleNewRound(round: bigint) {
|
|
158
|
-
this.log.info(`Starting new
|
|
149
|
+
this.log.info(`Starting new slashing round ${round}`);
|
|
159
150
|
await this.offensesCollector.handleNewRound(round);
|
|
160
151
|
}
|
|
161
152
|
|
|
162
|
-
/** Called when we see a RoundExecuted event on the
|
|
153
|
+
/** Called when we see a RoundExecuted event on the SlashingProposer (just for logging). */
|
|
163
154
|
protected async handleRoundExecuted(round: bigint, slashCount: bigint, l1BlockHash: Hex) {
|
|
164
155
|
const slashes = await this.rollup.getSlashEvents(l1BlockHash);
|
|
165
156
|
this.log.info(`Slashing round ${round} has been executed with ${slashCount} slashes`, { slashes });
|
|
@@ -240,7 +231,7 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
|
|
|
240
231
|
this.log.debug(`Testing if slashing round ${executableRound} is executable`, logData);
|
|
241
232
|
|
|
242
233
|
try {
|
|
243
|
-
const roundInfo = await this.
|
|
234
|
+
const roundInfo = await this.slashingProposer.getRound(executableRound);
|
|
244
235
|
logData = { ...logData, roundInfo };
|
|
245
236
|
if (roundInfo.isExecuted) {
|
|
246
237
|
this.log.verbose(`Round ${executableRound} has already been executed`, logData);
|
|
@@ -254,7 +245,7 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
|
|
|
254
245
|
}
|
|
255
246
|
|
|
256
247
|
// Check if round is ready to execute at the given slot
|
|
257
|
-
const isReadyToExecute = await this.
|
|
248
|
+
const isReadyToExecute = await this.slashingProposer.isRoundReadyToExecute(executableRound, slotNumber);
|
|
258
249
|
if (!isReadyToExecute) {
|
|
259
250
|
this.log.warn(
|
|
260
251
|
`Round ${executableRound} is not ready to execute at slot ${slotNumber} according to contract check`,
|
|
@@ -264,14 +255,14 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
|
|
|
264
255
|
}
|
|
265
256
|
|
|
266
257
|
// Check if the round yields any slashing at all
|
|
267
|
-
const { actions: slashActions, committees } = await this.
|
|
258
|
+
const { actions: slashActions, committees } = await this.slashingProposer.getTally(executableRound);
|
|
268
259
|
if (slashActions.length === 0) {
|
|
269
260
|
this.log.verbose(`Round ${executableRound} does not resolve in any slashing`, logData);
|
|
270
261
|
return undefined;
|
|
271
262
|
}
|
|
272
263
|
|
|
273
264
|
// Check if the slash payload is vetoed
|
|
274
|
-
const payload = await this.
|
|
265
|
+
const payload = await this.slashingProposer.getPayload(executableRound);
|
|
275
266
|
const isVetoed = await this.slasher.isPayloadVetoed(payload.address);
|
|
276
267
|
if (isVetoed) {
|
|
277
268
|
this.log.warn(`Round ${executableRound} payload is vetoed (skipping execution)`, {
|
|
@@ -281,8 +272,12 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
|
|
|
281
272
|
return undefined;
|
|
282
273
|
}
|
|
283
274
|
|
|
275
|
+
const slashActionsWithAmounts = slashActions.map(action => ({
|
|
276
|
+
validator: action.validator.toString(),
|
|
277
|
+
slashAmount: action.slashAmount.toString(),
|
|
278
|
+
}));
|
|
284
279
|
this.log.info(`Round ${executableRound} is ready to execute with ${slashActions.length} slashes`, {
|
|
285
|
-
slashActions,
|
|
280
|
+
slashActions: slashActionsWithAmounts,
|
|
286
281
|
payloadAddress: payload.address.toString(),
|
|
287
282
|
...logData,
|
|
288
283
|
});
|
|
@@ -357,11 +352,13 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
|
|
|
357
352
|
|
|
358
353
|
const committees = await this.collectCommitteesActiveDuringRound(slashedRound);
|
|
359
354
|
const epochsForCommittees = getEpochsForRound(slashedRound, this.settings);
|
|
355
|
+
const { slashMaxPayloadSize } = this.config;
|
|
360
356
|
const votes = getSlashConsensusVotesFromOffenses(
|
|
361
357
|
offensesToSlash,
|
|
362
358
|
committees,
|
|
363
359
|
epochsForCommittees.map(e => BigInt(e)),
|
|
364
|
-
this.settings,
|
|
360
|
+
{ ...this.settings, maxSlashedValidators: slashMaxPayloadSize },
|
|
361
|
+
this.log,
|
|
365
362
|
);
|
|
366
363
|
if (votes.every(v => v === 0)) {
|
|
367
364
|
this.log.warn(`Computed votes for offenses are all zero. Skipping vote.`, {
|
|
@@ -399,17 +396,9 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
|
|
|
399
396
|
);
|
|
400
397
|
}
|
|
401
398
|
|
|
402
|
-
/**
|
|
403
|
-
* Get slash payloads is NOT SUPPORTED in tally model
|
|
404
|
-
* @throws Error indicating this operation is not supported
|
|
405
|
-
*/
|
|
406
|
-
public getSlashPayloads(): Promise<SlashPayloadRound[]> {
|
|
407
|
-
return Promise.reject(new Error('Tally slashing model does not support slash payloads'));
|
|
408
|
-
}
|
|
409
|
-
|
|
410
399
|
/**
|
|
411
400
|
* Gather offenses to be slashed on a given round.
|
|
412
|
-
*
|
|
401
|
+
* Round N slashes validators from round N - slashOffsetInRounds.
|
|
413
402
|
* @param round - The round to get offenses for, defaults to current round
|
|
414
403
|
* @returns Array of pending offenses for the round with offset applied
|
|
415
404
|
*/
|
|
@@ -422,9 +411,9 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
|
|
|
422
411
|
return await this.offensesStore.getOffensesForRound(targetRound);
|
|
423
412
|
}
|
|
424
413
|
|
|
425
|
-
/** Returns all
|
|
426
|
-
public
|
|
427
|
-
return this.offensesStore.
|
|
414
|
+
/** Returns all offenses stored */
|
|
415
|
+
public getOffenses(): Promise<Offense[]> {
|
|
416
|
+
return this.offensesStore.getOffenses();
|
|
428
417
|
}
|
|
429
418
|
|
|
430
419
|
/**
|
|
@@ -1,14 +1,13 @@
|
|
|
1
1
|
import { EpochCache } from '@aztec/epoch-cache';
|
|
2
|
-
import type { ViemClient } from '@aztec/ethereum';
|
|
3
2
|
import { RollupContract } from '@aztec/ethereum/contracts';
|
|
3
|
+
import type { ViemClient } from '@aztec/ethereum/types';
|
|
4
4
|
import type { SlotNumber } from '@aztec/foundation/branded-types';
|
|
5
|
-
import { EthAddress } from '@aztec/foundation/eth-address';
|
|
6
5
|
import { createLogger } from '@aztec/foundation/log';
|
|
7
6
|
import { DateProvider } from '@aztec/foundation/timer';
|
|
8
|
-
import type { DataStoreConfig } from '@aztec/kv-store/config';
|
|
9
7
|
import { AztecLMDBStoreV2 } from '@aztec/kv-store/lmdb-v2';
|
|
10
8
|
import type { SlasherConfig } from '@aztec/stdlib/interfaces/server';
|
|
11
|
-
import type {
|
|
9
|
+
import type { DataStoreConfig } from '@aztec/stdlib/kv-store';
|
|
10
|
+
import type { Offense, ProposerSlashAction } from '@aztec/stdlib/slashing';
|
|
12
11
|
|
|
13
12
|
import { createSlasherImplementation } from './factory/create_implementation.js';
|
|
14
13
|
import type { SlasherClientInterface } from './slasher_client_interface.js';
|
|
@@ -27,11 +26,11 @@ export class SlasherClientFacade implements SlasherClientInterface {
|
|
|
27
26
|
private config: SlasherConfig & DataStoreConfig & { ethereumSlotDuration: number },
|
|
28
27
|
private rollup: RollupContract,
|
|
29
28
|
private l1Client: ViemClient,
|
|
30
|
-
private slashFactoryAddress: EthAddress | undefined,
|
|
31
29
|
private watchers: Watcher[],
|
|
32
30
|
private epochCache: EpochCache,
|
|
33
31
|
private dateProvider: DateProvider,
|
|
34
32
|
private kvStore: AztecLMDBStoreV2,
|
|
33
|
+
private rollupRegisteredAtL2Slot: SlotNumber,
|
|
35
34
|
private logger = createLogger('slasher'),
|
|
36
35
|
) {}
|
|
37
36
|
|
|
@@ -62,16 +61,12 @@ export class SlasherClientFacade implements SlasherClientInterface {
|
|
|
62
61
|
this.watchers.forEach(watcher => watcher.updateConfig?.(config));
|
|
63
62
|
}
|
|
64
63
|
|
|
65
|
-
public getSlashPayloads(): Promise<SlashPayloadRound[]> {
|
|
66
|
-
return this.client?.getSlashPayloads() ?? Promise.reject(new Error('Slasher client not initialized'));
|
|
67
|
-
}
|
|
68
|
-
|
|
69
64
|
public gatherOffensesForRound(round?: bigint): Promise<Offense[]> {
|
|
70
65
|
return this.client?.gatherOffensesForRound(round) ?? Promise.reject(new Error('Slasher client not initialized'));
|
|
71
66
|
}
|
|
72
67
|
|
|
73
|
-
public
|
|
74
|
-
return this.client?.
|
|
68
|
+
public getOffenses(): Promise<Offense[]> {
|
|
69
|
+
return this.client?.getOffenses() ?? Promise.reject(new Error('Slasher client not initialized'));
|
|
75
70
|
}
|
|
76
71
|
|
|
77
72
|
public getProposerActions(slotNumber: SlotNumber): Promise<ProposerSlashAction[]> {
|
|
@@ -83,11 +78,11 @@ export class SlasherClientFacade implements SlasherClientInterface {
|
|
|
83
78
|
this.config,
|
|
84
79
|
this.rollup,
|
|
85
80
|
this.l1Client,
|
|
86
|
-
this.slashFactoryAddress,
|
|
87
81
|
this.watchers,
|
|
88
82
|
this.epochCache,
|
|
89
83
|
this.dateProvider,
|
|
90
84
|
this.kvStore,
|
|
85
|
+
this.rollupRegisteredAtL2Slot,
|
|
91
86
|
this.logger,
|
|
92
87
|
);
|
|
93
88
|
}
|
|
@@ -1,11 +1,8 @@
|
|
|
1
1
|
import type { SlotNumber } from '@aztec/foundation/branded-types';
|
|
2
2
|
import type { SlasherConfig } from '@aztec/stdlib/interfaces/server';
|
|
3
|
-
import type { Offense, ProposerSlashAction
|
|
3
|
+
import type { Offense, ProposerSlashAction } from '@aztec/stdlib/slashing';
|
|
4
4
|
|
|
5
|
-
/**
|
|
6
|
-
* Common interface for slasher clients used by the Aztec node.
|
|
7
|
-
* Both Empire and Consensus slasher clients implement this interface.
|
|
8
|
-
*/
|
|
5
|
+
/** Common interface for slasher clients used by the Aztec node. */
|
|
9
6
|
export interface SlasherClientInterface {
|
|
10
7
|
/** Start the slasher client */
|
|
11
8
|
start(): Promise<void>;
|
|
@@ -13,25 +10,13 @@ export interface SlasherClientInterface {
|
|
|
13
10
|
/** Stop the slasher client */
|
|
14
11
|
stop(): Promise<void>;
|
|
15
12
|
|
|
16
|
-
/**
|
|
17
|
-
* Get slash payloads for the Empire model.
|
|
18
|
-
* The Consensus model should throw an error when this is called.
|
|
19
|
-
*/
|
|
20
|
-
getSlashPayloads(): Promise<SlashPayloadRound[]>;
|
|
21
|
-
|
|
22
|
-
/**
|
|
23
|
-
* Gather offenses for a given round, defaults to current.
|
|
24
|
-
* Used by both Empire and Consensus models.
|
|
25
|
-
*/
|
|
13
|
+
/** Gather offenses for a given round, defaults to current. */
|
|
26
14
|
gatherOffensesForRound(round?: bigint): Promise<Offense[]>;
|
|
27
15
|
|
|
28
|
-
/** Returns all
|
|
29
|
-
|
|
16
|
+
/** Returns all offenses */
|
|
17
|
+
getOffenses(): Promise<Offense[]>;
|
|
30
18
|
|
|
31
|
-
/**
|
|
32
|
-
* Update the configuration.
|
|
33
|
-
* Used by both Empire and Consensus models.
|
|
34
|
-
*/
|
|
19
|
+
/** Update the configuration. */
|
|
35
20
|
updateConfig(config: Partial<SlasherConfig>): void;
|
|
36
21
|
|
|
37
22
|
/**
|
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
import { createLogger } from '@aztec/aztec.js/log';
|
|
2
|
-
import type { AztecAsyncKVStore, AztecAsyncMap, AztecAsyncMultiMap
|
|
2
|
+
import type { AztecAsyncKVStore, AztecAsyncMap, AztecAsyncMultiMap } from '@aztec/kv-store';
|
|
3
3
|
import {
|
|
4
4
|
type Offense,
|
|
5
5
|
type OffenseIdentifier,
|
|
@@ -14,10 +14,7 @@ export class SlasherOffensesStore {
|
|
|
14
14
|
/** Map from offense key to offense data */
|
|
15
15
|
private offenses: AztecAsyncMap<string, Buffer>;
|
|
16
16
|
|
|
17
|
-
/**
|
|
18
|
-
private offensesSlashed: AztecAsyncSet<string>;
|
|
19
|
-
|
|
20
|
-
/** Multimap from round to offense keys (only used for consensus based slashing) */
|
|
17
|
+
/** Multimap from round to offense keys */
|
|
21
18
|
private roundsOffenses: AztecAsyncMultiMap<string, string>;
|
|
22
19
|
|
|
23
20
|
private log = createLogger('slasher:store:offenses');
|
|
@@ -32,18 +29,13 @@ export class SlasherOffensesStore {
|
|
|
32
29
|
) {
|
|
33
30
|
this.offenses = kvStore.openMap('offenses');
|
|
34
31
|
this.roundsOffenses = kvStore.openMultiMap('rounds-offenses');
|
|
35
|
-
this.offensesSlashed = kvStore.openSet('offenses-slashed');
|
|
36
32
|
}
|
|
37
33
|
|
|
38
|
-
/** Returns all offenses
|
|
39
|
-
public async
|
|
34
|
+
/** Returns all offenses */
|
|
35
|
+
public async getOffenses(): Promise<Offense[]> {
|
|
40
36
|
const offenses: Offense[] = [];
|
|
41
|
-
for await (const [
|
|
42
|
-
|
|
43
|
-
continue; // Skip executed offenses
|
|
44
|
-
}
|
|
45
|
-
const offense = deserializeOffense(buffer);
|
|
46
|
-
offenses.push(offense);
|
|
37
|
+
for await (const [, buffer] of this.offenses.entriesAsync()) {
|
|
38
|
+
offenses.push(deserializeOffense(buffer));
|
|
47
39
|
}
|
|
48
40
|
return offenses;
|
|
49
41
|
}
|
|
@@ -61,35 +53,21 @@ export class SlasherOffensesStore {
|
|
|
61
53
|
return offenses;
|
|
62
54
|
}
|
|
63
55
|
|
|
64
|
-
/** Returns whether an offense is pending (ie not marked as slashed) */
|
|
65
|
-
public async hasPendingOffense(offense: OffenseIdentifier): Promise<boolean> {
|
|
66
|
-
const key = this.getOffenseKey(offense);
|
|
67
|
-
return (await this.offenses.getAsync(key)) !== undefined && !(await this.offensesSlashed.hasAsync(key));
|
|
68
|
-
}
|
|
69
|
-
|
|
70
56
|
/** Returns whether we have seen this offense */
|
|
71
57
|
public async hasOffense(offense: OffenseIdentifier): Promise<boolean> {
|
|
72
58
|
const key = this.getOffenseKey(offense);
|
|
73
59
|
return (await this.offenses.getAsync(key)) !== undefined;
|
|
74
60
|
}
|
|
75
61
|
|
|
76
|
-
/** Adds a new offense
|
|
77
|
-
public async
|
|
62
|
+
/** Adds a new offense */
|
|
63
|
+
public async addOffense(offense: Offense): Promise<void> {
|
|
78
64
|
const key = this.getOffenseKey(offense);
|
|
79
|
-
await this.offenses.set(key, serializeOffense(offense));
|
|
80
65
|
const round = getRoundForOffense(offense, this.settings);
|
|
81
|
-
await this.roundsOffenses.set(this.getRoundKey(round), key);
|
|
82
|
-
this.log.trace(`Adding pending offense ${key} for round ${round}`);
|
|
83
|
-
}
|
|
84
|
-
|
|
85
|
-
/** Marks the given offenses as slashed (regardless of whether they are known or not) */
|
|
86
|
-
public async markAsSlashed(offenses: OffenseIdentifier[]): Promise<void> {
|
|
87
66
|
await this.kvStore.transactionAsync(async () => {
|
|
88
|
-
|
|
89
|
-
|
|
90
|
-
await this.offensesSlashed.add(key);
|
|
91
|
-
}
|
|
67
|
+
await this.offenses.set(key, serializeOffense(offense));
|
|
68
|
+
await this.roundsOffenses.set(this.getRoundKey(round), key);
|
|
92
69
|
});
|
|
70
|
+
this.log.trace(`Adding pending offense ${key} for round ${round}`);
|
|
93
71
|
}
|
|
94
72
|
|
|
95
73
|
/** Prunes all offenses expired from the store */
|
|
@@ -123,7 +101,6 @@ export class SlasherOffensesStore {
|
|
|
123
101
|
for (const key of expiredOffenseKeys) {
|
|
124
102
|
this.log.trace(`Deleting offense ${key}`);
|
|
125
103
|
await this.offenses.delete(key);
|
|
126
|
-
await this.offensesSlashed.delete(key);
|
|
127
104
|
}
|
|
128
105
|
for (const roundKey of expiredRoundKeys) {
|
|
129
106
|
this.log.trace(`Deleting round info for ${roundKey}`);
|
package/src/watcher.ts
CHANGED
|
@@ -10,7 +10,7 @@ export interface WantToSlashArgs {
|
|
|
10
10
|
validator: EthAddress;
|
|
11
11
|
amount: bigint;
|
|
12
12
|
offenseType: OffenseType;
|
|
13
|
-
epochOrSlot: bigint; // Epoch number for epoch-based offenses,
|
|
13
|
+
epochOrSlot: bigint; // Epoch number for epoch-based offenses, slot number for slot-based
|
|
14
14
|
}
|
|
15
15
|
|
|
16
16
|
// Event map for specific, known events of a watcher
|
|
@@ -3,12 +3,12 @@ import { SlotNumber } from '@aztec/foundation/branded-types';
|
|
|
3
3
|
import { merge, pick } from '@aztec/foundation/collection';
|
|
4
4
|
import { type Logger, createLogger } from '@aztec/foundation/log';
|
|
5
5
|
import {
|
|
6
|
-
type
|
|
7
|
-
type L2BlockInfo,
|
|
6
|
+
type InvalidCheckpointDetectedEvent,
|
|
8
7
|
type L2BlockSourceEventEmitter,
|
|
9
8
|
L2BlockSourceEvents,
|
|
10
|
-
type
|
|
9
|
+
type ValidateCheckpointNegativeResult,
|
|
11
10
|
} from '@aztec/stdlib/block';
|
|
11
|
+
import type { CheckpointInfo } from '@aztec/stdlib/checkpoint';
|
|
12
12
|
import { OffenseType } from '@aztec/stdlib/slashing';
|
|
13
13
|
|
|
14
14
|
import EventEmitter from 'node:events';
|
|
@@ -32,19 +32,19 @@ type AttestationsBlockWatcherConfig = Pick<SlasherConfig, (typeof AttestationsBl
|
|
|
32
32
|
export class AttestationsBlockWatcher extends (EventEmitter as new () => WatcherEmitter) implements Watcher {
|
|
33
33
|
private log: Logger = createLogger('attestations-block-watcher');
|
|
34
34
|
|
|
35
|
-
// Only keep track of the last N invalid
|
|
36
|
-
private
|
|
35
|
+
// Only keep track of the last N invalid checkpoints
|
|
36
|
+
private maxInvalidCheckpoints = 100;
|
|
37
37
|
|
|
38
38
|
// All invalid archive roots seen
|
|
39
39
|
private invalidArchiveRoots: Set<string> = new Set();
|
|
40
40
|
|
|
41
41
|
private config: AttestationsBlockWatcherConfig;
|
|
42
42
|
|
|
43
|
-
private
|
|
43
|
+
private boundHandleInvalidCheckpoint = (event: InvalidCheckpointDetectedEvent) => {
|
|
44
44
|
try {
|
|
45
|
-
this.
|
|
45
|
+
this.handleInvalidCheckpoint(event);
|
|
46
46
|
} catch (err) {
|
|
47
|
-
this.log.error('Error handling invalid
|
|
47
|
+
this.log.error('Error handling invalid checkpoint', err, {
|
|
48
48
|
...event.validationResult,
|
|
49
49
|
reason: event.validationResult.reason,
|
|
50
50
|
});
|
|
@@ -67,54 +67,61 @@ export class AttestationsBlockWatcher extends (EventEmitter as new () => Watcher
|
|
|
67
67
|
}
|
|
68
68
|
|
|
69
69
|
public start() {
|
|
70
|
-
this.l2BlockSource.on(
|
|
70
|
+
this.l2BlockSource.events.on(
|
|
71
|
+
L2BlockSourceEvents.InvalidAttestationsCheckpointDetected,
|
|
72
|
+
this.boundHandleInvalidCheckpoint,
|
|
73
|
+
);
|
|
71
74
|
return Promise.resolve();
|
|
72
75
|
}
|
|
73
76
|
|
|
74
77
|
public stop() {
|
|
75
|
-
this.l2BlockSource.removeListener(
|
|
76
|
-
L2BlockSourceEvents.
|
|
77
|
-
this.
|
|
78
|
+
this.l2BlockSource.events.removeListener(
|
|
79
|
+
L2BlockSourceEvents.InvalidAttestationsCheckpointDetected,
|
|
80
|
+
this.boundHandleInvalidCheckpoint,
|
|
78
81
|
);
|
|
79
82
|
return Promise.resolve();
|
|
80
83
|
}
|
|
81
84
|
|
|
82
|
-
|
|
85
|
+
/** Event handler for invalid checkpoints as reported by the archiver. Public for testing purposes. */
|
|
86
|
+
public handleInvalidCheckpoint(event: InvalidCheckpointDetectedEvent): void {
|
|
83
87
|
const { validationResult } = event;
|
|
84
|
-
const
|
|
88
|
+
const checkpoint = validationResult.checkpoint;
|
|
85
89
|
|
|
86
|
-
// Check if we already have processed this
|
|
87
|
-
if (this.invalidArchiveRoots.has(
|
|
88
|
-
this.log.trace(`Already processed invalid
|
|
90
|
+
// Check if we already have processed this checkpoint, archiver may emit the same event multiple times
|
|
91
|
+
if (this.invalidArchiveRoots.has(checkpoint.archive.toString())) {
|
|
92
|
+
this.log.trace(`Already processed invalid checkpoint ${checkpoint.checkpointNumber}`);
|
|
89
93
|
return;
|
|
90
94
|
}
|
|
91
95
|
|
|
92
|
-
this.log.verbose(`Detected invalid
|
|
93
|
-
...
|
|
96
|
+
this.log.verbose(`Detected invalid checkpoint ${checkpoint.checkpointNumber}`, {
|
|
97
|
+
...checkpoint,
|
|
94
98
|
reason: validationResult.valid === false ? validationResult.reason : 'unknown',
|
|
95
99
|
});
|
|
96
100
|
|
|
97
|
-
// Store the invalid
|
|
98
|
-
this.
|
|
101
|
+
// Store the invalid checkpoint
|
|
102
|
+
this.addInvalidCheckpoint(event.validationResult.checkpoint);
|
|
99
103
|
|
|
100
|
-
// Slash the proposer of the invalid
|
|
104
|
+
// Slash the proposer of the invalid checkpoint
|
|
101
105
|
this.slashProposer(event.validationResult);
|
|
102
106
|
|
|
103
|
-
// Check if the parent of this
|
|
107
|
+
// Check if the parent of this checkpoint is invalid as well, if so, we will slash its attestors as well
|
|
104
108
|
this.slashAttestorsOnAncestorInvalid(event.validationResult);
|
|
105
109
|
}
|
|
106
110
|
|
|
107
|
-
private slashAttestorsOnAncestorInvalid(validationResult:
|
|
108
|
-
const
|
|
111
|
+
private slashAttestorsOnAncestorInvalid(validationResult: ValidateCheckpointNegativeResult) {
|
|
112
|
+
const checkpoint = validationResult.checkpoint;
|
|
109
113
|
|
|
110
|
-
const parentArchive =
|
|
114
|
+
const parentArchive = checkpoint.lastArchive.toString();
|
|
111
115
|
if (this.invalidArchiveRoots.has(parentArchive)) {
|
|
112
116
|
const attestors = validationResult.attestors;
|
|
113
|
-
this.log.info(
|
|
114
|
-
|
|
115
|
-
|
|
116
|
-
|
|
117
|
-
|
|
117
|
+
this.log.info(
|
|
118
|
+
`Want to slash attestors of checkpoint ${checkpoint.checkpointNumber} built on invalid checkpoint`,
|
|
119
|
+
{
|
|
120
|
+
...checkpoint,
|
|
121
|
+
...attestors,
|
|
122
|
+
parentArchive,
|
|
123
|
+
},
|
|
124
|
+
);
|
|
118
125
|
|
|
119
126
|
this.emit(
|
|
120
127
|
WANT_TO_SLASH_EVENT,
|
|
@@ -122,20 +129,26 @@ export class AttestationsBlockWatcher extends (EventEmitter as new () => Watcher
|
|
|
122
129
|
validator: attestor,
|
|
123
130
|
amount: this.config.slashAttestDescendantOfInvalidPenalty,
|
|
124
131
|
offenseType: OffenseType.ATTESTED_DESCENDANT_OF_INVALID,
|
|
125
|
-
epochOrSlot: BigInt(SlotNumber(
|
|
132
|
+
epochOrSlot: BigInt(SlotNumber(checkpoint.slotNumber)),
|
|
126
133
|
})),
|
|
127
134
|
);
|
|
128
135
|
}
|
|
129
136
|
}
|
|
130
137
|
|
|
131
|
-
private slashProposer(validationResult:
|
|
132
|
-
const { reason,
|
|
133
|
-
const
|
|
134
|
-
const slot =
|
|
135
|
-
const
|
|
138
|
+
private slashProposer(validationResult: ValidateCheckpointNegativeResult) {
|
|
139
|
+
const { reason, checkpoint } = validationResult;
|
|
140
|
+
const checkpointNumber = checkpoint.checkpointNumber;
|
|
141
|
+
const slot = checkpoint.slotNumber;
|
|
142
|
+
const epochCommitteeInfo = {
|
|
143
|
+
committee: validationResult.committee,
|
|
144
|
+
seed: validationResult.seed,
|
|
145
|
+
epoch: validationResult.epoch,
|
|
146
|
+
isEscapeHatchOpen: false,
|
|
147
|
+
};
|
|
148
|
+
const proposer = this.epochCache.getProposerFromEpochCommittee(epochCommitteeInfo, slot);
|
|
136
149
|
|
|
137
150
|
if (!proposer) {
|
|
138
|
-
this.log.warn(`No proposer found for
|
|
151
|
+
this.log.warn(`No proposer found for checkpoint ${checkpointNumber} at slot ${slot}`);
|
|
139
152
|
return;
|
|
140
153
|
}
|
|
141
154
|
|
|
@@ -148,15 +161,15 @@ export class AttestationsBlockWatcher extends (EventEmitter as new () => Watcher
|
|
|
148
161
|
epochOrSlot: BigInt(slot),
|
|
149
162
|
};
|
|
150
163
|
|
|
151
|
-
this.log.info(`Want to slash proposer of
|
|
152
|
-
...
|
|
164
|
+
this.log.info(`Want to slash proposer of checkpoint ${checkpointNumber} due to ${reason}`, {
|
|
165
|
+
...checkpoint,
|
|
153
166
|
...args,
|
|
154
167
|
});
|
|
155
168
|
|
|
156
169
|
this.emit(WANT_TO_SLASH_EVENT, [args]);
|
|
157
170
|
}
|
|
158
171
|
|
|
159
|
-
private getOffenseFromInvalidationReason(reason:
|
|
172
|
+
private getOffenseFromInvalidationReason(reason: ValidateCheckpointNegativeResult['reason']): OffenseType {
|
|
160
173
|
switch (reason) {
|
|
161
174
|
case 'invalid-attestation':
|
|
162
175
|
return OffenseType.PROPOSED_INCORRECT_ATTESTATIONS;
|
|
@@ -169,11 +182,11 @@ export class AttestationsBlockWatcher extends (EventEmitter as new () => Watcher
|
|
|
169
182
|
}
|
|
170
183
|
}
|
|
171
184
|
|
|
172
|
-
private
|
|
173
|
-
this.invalidArchiveRoots.add(
|
|
185
|
+
private addInvalidCheckpoint(checkpoint: CheckpointInfo) {
|
|
186
|
+
this.invalidArchiveRoots.add(checkpoint.archive.toString());
|
|
174
187
|
|
|
175
188
|
// Prune old entries if we exceed the maximum
|
|
176
|
-
if (this.invalidArchiveRoots.size > this.
|
|
189
|
+
if (this.invalidArchiveRoots.size > this.maxInvalidCheckpoints) {
|
|
177
190
|
const oldestKey = this.invalidArchiveRoots.keys().next().value!;
|
|
178
191
|
this.invalidArchiveRoots.delete(oldestKey);
|
|
179
192
|
}
|