Skip to content

Commit 498ab18

Browse files
JJSantos22pandaio22
authored andcommitted
fix(satp-hermes): resolve proto files import cycles closes #3773 (#3834)
Signed-off-by: Jorge Santos <[email protected]>
1 parent 3423f2d commit 498ab18

32 files changed

+201
-202
lines changed

packages/cactus-plugin-satp-hermes/src/main/proto/cacti/satp/v02/common/session.proto packages/cactus-plugin-satp-hermes/src/main/proto/cacti/satp/v02/session/session.proto

+10-11
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,11 @@
11
syntax = "proto3";
2-
package cacti.satp.v02.common;
2+
package cacti.satp.v02.session;
33

44
import "cacti/satp/v02/common/message.proto";
55
import "cacti/satp/v02/service/stage_0.proto";
66
import "cacti/satp/v02/service/stage_1.proto";
77
import "cacti/satp/v02/service/stage_2.proto";
88
import "cacti/satp/v02/service/stage_3.proto";
9-
109
import "google/protobuf/empty.proto";
1110

1211
message SessionData {
@@ -245,6 +244,14 @@ message Stage3Timestamps {
245244
string transfer_complete_response_message_timestamp = 6;
246245
}
247246

247+
enum SATPStage {
248+
SATP_STAGE_UNSPECIFIED = 0;
249+
SATP_STAGE_0 = 1;
250+
SATP_STAGE_1 = 2;
251+
SATP_STAGE_2 = 3;
252+
SATP_STAGE_3 = 4;
253+
}
254+
248255
message SendStatusRequest {
249256
string status = 1;
250257
bool has_backup = 2;
@@ -255,14 +262,6 @@ message GetStatusResponse {
255262
bool has_backup = 2;
256263
}
257264

258-
enum SATPStage {
259-
SATP_STAGE_UNSPECIFIED = 0;
260-
SATP_STAGE_0 = 1;
261-
SATP_STAGE_1 = 2;
262-
SATP_STAGE_2 = 3;
263-
SATP_STAGE_3 = 4;
264-
}
265-
266265
message PingResponse {
267266
string message = 1;
268267
}
@@ -285,4 +284,4 @@ service CommonService {
285284
rpc Ping(google.protobuf.Empty) returns (PingResponse) {}
286285
rpc Rollback(google.protobuf.Empty) returns (RollbackResponse) {}
287286
rpc GetStageVersion(google.protobuf.Empty) returns (GetStageVersionResponse) {};
288-
}
287+
}

packages/cactus-plugin-satp-hermes/src/main/typescript/api1/admin/get-status-handler-service.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ import {
1414
getStageName,
1515
getStateName,
1616
} from "../../core/session-utils";
17-
import { State } from "../../generated/proto/cacti/satp/v02/common/session_pb";
17+
import { State } from "../../generated/proto/cacti/satp/v02/session/session_pb";
1818
import { LedgerType } from "@hyperledger/cactus-core-api";
1919

2020
export async function executeGetStatus(

packages/cactus-plugin-satp-hermes/src/main/typescript/core/crash-management/client-service.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import { JsObjectSigner, Logger } from "@hyperledger/cactus-common";
1111
import { create } from "@bufbuild/protobuf";
1212
import { stringify as safeStableStringify } from "safe-stable-stringify";
1313
import { bufArray2HexStr, sign } from "../../gateway-utils";
14-
import type { SessionData } from "../../generated/proto/cacti/satp/v02/common/session_pb";
14+
import type { SessionData } from "../../generated/proto/cacti/satp/v02/session/session_pb";
1515
import { getCrashedStage } from "../session-utils";
1616

1717
export class CrashRecoveryClientService {

packages/cactus-plugin-satp-hermes/src/main/typescript/core/crash-management/crash-handler.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import type {
1515
RollbackState,
1616
} from "../../generated/proto/cacti/satp/v02/service/crash_recovery_pb";
1717
import { type SATPHandler, SATPHandlerType } from "../../types/satp-protocol";
18-
import type { SessionData } from "../../generated/proto/cacti/satp/v02/common/session_pb";
18+
import type { SessionData } from "../../generated/proto/cacti/satp/v02/session/session_pb";
1919

2020
export class CrashRecoveryHandler implements SATPHandler {
2121
private readonly log: Logger;

packages/cactus-plugin-satp-hermes/src/main/typescript/core/crash-management/rollback/rollback-strategy-factory.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import {
1010
type Type,
1111
SATPStage,
1212
type SessionData,
13-
} from "../../../generated/proto/cacti/satp/v02/common/session_pb";
13+
} from "../../../generated/proto/cacti/satp/v02/session/session_pb";
1414
import { getCrashedStage } from "../../session-utils";
1515

1616
// TODO: fix for single-gateway setups to handle both client and server data together

packages/cactus-plugin-satp-hermes/src/main/typescript/core/crash-management/rollback/stage0-rollback-strategy.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import {
1212
Type,
1313
SATPStage,
1414
type SessionData,
15-
} from "../../../generated/proto/cacti/satp/v02/common/session_pb";
15+
} from "../../../generated/proto/cacti/satp/v02/session/session_pb";
1616

1717
export class Stage0RollbackStrategy implements RollbackStrategy {
1818
private log: Logger;

packages/cactus-plugin-satp-hermes/src/main/typescript/core/crash-management/rollback/stage1-rollback-strategy.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import {
1111
Type,
1212
SATPStage,
1313
type SessionData,
14-
} from "../../../generated/proto/cacti/satp/v02/common/session_pb";
14+
} from "../../../generated/proto/cacti/satp/v02/session/session_pb";
1515

1616
export class Stage1RollbackStrategy implements RollbackStrategy {
1717
private log: Logger;

packages/cactus-plugin-satp-hermes/src/main/typescript/core/crash-management/rollback/stage2-rollback-strategy.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import {
1212
Type,
1313
SATPStage,
1414
type SessionData,
15-
} from "../../../generated/proto/cacti/satp/v02/common/session_pb";
15+
} from "../../../generated/proto/cacti/satp/v02/session/session_pb";
1616

1717
export class Stage2RollbackStrategy implements RollbackStrategy {
1818
private log: Logger;

packages/cactus-plugin-satp-hermes/src/main/typescript/core/crash-management/rollback/stage3-rollback-strategy.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import {
1212
Type,
1313
SATPStage,
1414
type SessionData,
15-
} from "../../../generated/proto/cacti/satp/v02/common/session_pb";
15+
} from "../../../generated/proto/cacti/satp/v02/session/session_pb";
1616

1717
export class Stage3RollbackStrategy implements RollbackStrategy {
1818
private log: Logger;

packages/cactus-plugin-satp-hermes/src/main/typescript/core/crash-management/server-service.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ import { create } from "@bufbuild/protobuf";
1818
import { stringify as safeStableStringify } from "safe-stable-stringify";
1919
import { bufArray2HexStr, sign, verifySignature } from "../../gateway-utils";
2020
import { SignatureVerificationError } from "../errors/satp-service-errors";
21-
import { Type } from "../../generated/proto/cacti/satp/v02/common/session_pb";
21+
import { Type } from "../../generated/proto/cacti/satp/v02/session/session_pb";
2222

2323
export class CrashRecoveryServerService {
2424
constructor(

packages/cactus-plugin-satp-hermes/src/main/typescript/core/satp-session.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ import {
3434
Stage3SignaturesSchema,
3535
Stage3TimestampsSchema,
3636
State,
37-
} from "../generated/proto/cacti/satp/v02/common/session_pb";
37+
} from "../generated/proto/cacti/satp/v02/session/session_pb";
3838
import {
3939
AccessControlProfileError,
4040
ClientGatewayPubkeyError,

packages/cactus-plugin-satp-hermes/src/main/typescript/core/session-utils.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import {
1212
SATPStage,
1313
SessionData,
1414
State,
15-
} from "../generated/proto/cacti/satp/v02/common/session_pb";
15+
} from "../generated/proto/cacti/satp/v02/session/session_pb";
1616
import {
1717
NewSessionRequest,
1818
NewSessionRequestSchema,

packages/cactus-plugin-satp-hermes/src/main/typescript/core/stage-services/client/stage1-client-service.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ import {
2929
ISATPServiceOptions,
3030
} from "../satp-service";
3131
import { commonBodyVerifier, signatureVerifier } from "../data-verifier";
32-
import { State } from "../../../generated/proto/cacti/satp/v02/common/session_pb";
32+
import { State } from "../../../generated/proto/cacti/satp/v02/session/session_pb";
3333
import {
3434
HashError,
3535
MessageTypeError,

packages/cactus-plugin-satp-hermes/src/main/typescript/core/stage-services/client/stage3-client-service.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ import {
4444
TokenIdMissingError,
4545
} from "../../errors/satp-service-errors";
4646
import { FailedToProcessError } from "../../errors/satp-handler-errors";
47-
import { State } from "../../../generated/proto/cacti/satp/v02/common/session_pb";
47+
import { State } from "../../../generated/proto/cacti/satp/v02/session/session_pb";
4848
import { create } from "@bufbuild/protobuf";
4949

5050
export class Stage3ClientService extends SATPService {

packages/cactus-plugin-satp-hermes/src/main/typescript/core/stage-services/data-verifier.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import {
66
} from "../../generated/proto/cacti/satp/v02/common/message_pb";
77
import { stringify as safeStableStringify } from "safe-stable-stringify";
88

9-
import { SessionData } from "../../generated/proto/cacti/satp/v02/common/session_pb";
9+
import { SessionData } from "../../generated/proto/cacti/satp/v02/session/session_pb";
1010
import { SATP_VERSION } from "../constants";
1111
import {
1212
ClientGatewayPubkeyError,

packages/cactus-plugin-satp-hermes/src/main/typescript/core/stage-services/server/stage1-server-service.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ import {
4343
} from "../../errors/satp-service-errors";
4444
import { SATPInternalError } from "../../errors/satp-errors";
4545
import { SessionNotFoundError } from "../../errors/satp-handler-errors";
46-
import { State } from "../../../generated/proto/cacti/satp/v02/common/session_pb";
46+
import { State } from "../../../generated/proto/cacti/satp/v02/session/session_pb";
4747
import { create } from "@bufbuild/protobuf";
4848
import { NetworkId } from "../../../services/network-identification/chainid-list";
4949
export class Stage1ServerService extends SATPService {

packages/cactus-plugin-satp-hermes/src/main/typescript/core/stage-services/server/stage3-server-service.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ import {
4949
SessionNotFoundError,
5050
} from "../../errors/satp-handler-errors";
5151
import { SATPInternalError } from "../../errors/satp-errors";
52-
import { State } from "../../../generated/proto/cacti/satp/v02/common/session_pb";
52+
import { State } from "../../../generated/proto/cacti/satp/v02/session/session_pb";
5353
import { create } from "@bufbuild/protobuf";
5454

5555
export class Stage3ServerService extends SATPService {

packages/cactus-plugin-satp-hermes/src/main/typescript/generated/proto/cacti/satp/v02/common/session_connect.ts packages/cactus-plugin-satp-hermes/src/main/typescript/generated/proto/cacti/satp/v02/session/session_connect.ts

+10-10
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,19 @@
11
// @generated by protoc-gen-connect-es v1.6.1 with parameter "target=ts,js_import_style=module"
2-
// @generated from file cacti/satp/v02/common/session.proto (package cacti.satp.v02.common, syntax proto3)
2+
// @generated from file cacti/satp/v02/session/session.proto (package cacti.satp.v02.session, syntax proto3)
33
/* eslint-disable */
44
// @ts-nocheck
55

66
import { Empty, MethodKind } from "@bufbuild/protobuf";
77
import { GetStageVersionResponse, GetStatusResponse, PingResponse, RollbackResponse, SendStatusRequest } from "./session_pb.js";
88

99
/**
10-
* @generated from service cacti.satp.v02.common.SessionStatusService
10+
* @generated from service cacti.satp.v02.session.SessionStatusService
1111
*/
1212
export const SessionStatusService = {
13-
typeName: "cacti.satp.v02.common.SessionStatusService",
13+
typeName: "cacti.satp.v02.session.SessionStatusService",
1414
methods: {
1515
/**
16-
* @generated from rpc cacti.satp.v02.common.SessionStatusService.GetStatus
16+
* @generated from rpc cacti.satp.v02.session.SessionStatusService.GetStatus
1717
*/
1818
getStatus: {
1919
name: "GetStatus",
@@ -22,7 +22,7 @@ export const SessionStatusService = {
2222
kind: MethodKind.Unary,
2323
},
2424
/**
25-
* @generated from rpc cacti.satp.v02.common.SessionStatusService.SendStatus
25+
* @generated from rpc cacti.satp.v02.session.SessionStatusService.SendStatus
2626
*/
2727
sendStatus: {
2828
name: "SendStatus",
@@ -36,13 +36,13 @@ export const SessionStatusService = {
3636
/**
3737
* TODO: define common RPC methods for each step. This is a draft
3838
*
39-
* @generated from service cacti.satp.v02.common.CommonService
39+
* @generated from service cacti.satp.v02.session.CommonService
4040
*/
4141
export const CommonService = {
42-
typeName: "cacti.satp.v02.common.CommonService",
42+
typeName: "cacti.satp.v02.session.CommonService",
4343
methods: {
4444
/**
45-
* @generated from rpc cacti.satp.v02.common.CommonService.Ping
45+
* @generated from rpc cacti.satp.v02.session.CommonService.Ping
4646
*/
4747
ping: {
4848
name: "Ping",
@@ -51,7 +51,7 @@ export const CommonService = {
5151
kind: MethodKind.Unary,
5252
},
5353
/**
54-
* @generated from rpc cacti.satp.v02.common.CommonService.Rollback
54+
* @generated from rpc cacti.satp.v02.session.CommonService.Rollback
5555
*/
5656
rollback: {
5757
name: "Rollback",
@@ -60,7 +60,7 @@ export const CommonService = {
6060
kind: MethodKind.Unary,
6161
},
6262
/**
63-
* @generated from rpc cacti.satp.v02.common.CommonService.GetStageVersion
63+
* @generated from rpc cacti.satp.v02.session.CommonService.GetStageVersion
6464
*/
6565
getStageVersion: {
6666
name: "GetStageVersion",

0 commit comments

Comments
 (0)