Skip to content

Commit

Permalink
feat(core-api): hasTransactionFinality() on connector API #354
Browse files Browse the repository at this point in the history
Primary change(s):
------------------

1. Add a feature of asking the connector at runtime whether its ledger has transaction finality guaranteed or not

Fixes: #354

Signed-off-by: jscode017 <[email protected]>
  • Loading branch information
jscode017 authored and petermetz committed Apr 29, 2021
1 parent 9f37755 commit a2d0582
Show file tree
Hide file tree
Showing 10 changed files with 106 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,10 @@ import {
ICactusPluginOptions,
} from "@hyperledger/cactus-core-api";

import { PluginRegistry } from "@hyperledger/cactus-core";
import {
PluginRegistry,
consensusHasTransactionFinality,
} from "@hyperledger/cactus-core";

import {
Checks,
Expand Down Expand Up @@ -128,7 +131,11 @@ export class PluginLedgerConnectorStub
> {
return ConsensusAlgorithmFamily.AUTHORITY;
}
public async hasTransactionFinality(): Promise<boolean> {
const currentConsensusAlgorithmFamily = await this.getConsensusAlgorithmFamily();

return consensusHasTransactionFinality(currentConsensusAlgorithmFamily);
}
public async transact(req: unknown): Promise<unknown> {
const fnTag = `${this.className}#transact()`;
Checks.truthy(req, `${fnTag} req`);
Expand Down
18 changes: 18 additions & 0 deletions packages/cactus-core-api/src/main/json/openapi.json
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,24 @@
"org.hyperledger.cactus.consensusalgorithm.PROOF_OF_WORK"
]
},
"ConsensusAlgorithmFamiliesWithTxFinality":{
"type": "string",
"description": "Enumerates a list of consensus algorithm families that provide immediate finality",
"enum": [
"org.hyperledger.cactus.consensusalgorithm.PROOF_OF_AUTHORITY",
"org.hyperledger.cactus.consensusalgorithm.PROOF_OF_STAKE"
]

},
"ConsensusAlgorithmFamiliesWithOutTxFinality":{
"description":"Enumerates a list of consensus algorithm families that do not provide immediate finality",
"type": "string",
"enum": [
"org.hyperledger.cactus.consensusalgorithm.PROOF_OF_WORK"
],
"x-enum-varnames": ["WORK"]

},
"PrimaryKey": {
"type": "string",
"minLength": 1,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -124,6 +124,25 @@ export interface CactusNodeMeta {
*/
publicKeyPem: string;
}
/**
* Enumerates a list of consensus algorithm families that do not provide immediate finality
* @export
* @enum {string}
*/
export enum ConsensusAlgorithmFamiliesWithOutTxFinality {
WORK = 'org.hyperledger.cactus.consensusalgorithm.PROOF_OF_WORK'
}

/**
* Enumerates a list of consensus algorithm families that provide immediate finality
* @export
* @enum {string}
*/
export enum ConsensusAlgorithmFamiliesWithTxFinality {
AUTHORITY = 'org.hyperledger.cactus.consensusalgorithm.PROOF_OF_AUTHORITY',
STAKE = 'org.hyperledger.cactus.consensusalgorithm.PROOF_OF_STAKE'
}

/**
* Enumerates a list of consensus algorithm families in existence. Does not intend to be an exhaustive list, just a practical one, meaning that we only include items here that are relevant to Hyperledger Cactus in fulfilling its own duties. This can be extended later as more sophisticated features of Cactus get implemented. This enum is meant to be first and foremest a useful abstraction for achieving practical tasks, not an encyclopedia and therefore we ask of everyone that this to be extended only in ways that serve a practical purpose for the runtime behavior of Cactus or Cactus plugins in general. The bottom line is that we can accept this enum being not 100% accurate as long as it 100% satisfies what it was designed to do.
* @export
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,4 +30,5 @@ export interface IPluginLedgerConnector<
* @see {ConsensusAlgorithmFamily}
*/
getConsensusAlgorithmFamily(): Promise<ConsensusAlgorithmFamily>;
hasTransactionFinality(): Promise<boolean>;
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
import {
ConsensusAlgorithmFamily,
ConsensusAlgorithmFamiliesWithTxFinality,
ConsensusAlgorithmFamiliesWithOutTxFinality,
} from "@hyperledger/cactus-core-api";

export function consensusHasTransactionFinality(
consensusAlgorithmFamily: ConsensusAlgorithmFamily,
): boolean {
const isInConsensusAlgorithmFamiliesWithTxFinality = (Object.values(
ConsensusAlgorithmFamiliesWithTxFinality,
) as string[]).includes(consensusAlgorithmFamily.toString());

const isInConsensusAlgorithmFamiliesWithOutTxFinality = (Object.values(
ConsensusAlgorithmFamiliesWithOutTxFinality,
) as string[]).includes(consensusAlgorithmFamily.toString());

const unrecognizedConsensusAlgorithmFamily =
!isInConsensusAlgorithmFamiliesWithTxFinality &&
!isInConsensusAlgorithmFamiliesWithOutTxFinality;

if (unrecognizedConsensusAlgorithmFamily) {
throw new Error(
`Unrecognized consensus algorithm family: ${consensusAlgorithmFamily}`,
);
}
return isInConsensusAlgorithmFamiliesWithTxFinality;
}
2 changes: 2 additions & 0 deletions packages/cactus-core/src/main/typescript/public-api.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,3 +9,5 @@ export {
AuthorizationOptionsProvider,
IEndpointAuthzOptionsProviderOptions,
} from "./web-services/authorization-options-provider";

export { consensusHasTransactionFinality } from "./consensus-has-transaction-finality";
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,10 @@ import {
IPluginKeychain,
} from "@hyperledger/cactus-core-api";

import { PluginRegistry } from "@hyperledger/cactus-core";
import {
PluginRegistry,
consensusHasTransactionFinality,
} from "@hyperledger/cactus-core";

import {
Checks,
Expand Down Expand Up @@ -221,7 +224,11 @@ export class PluginLedgerConnectorBesu
> {
return ConsensusAlgorithmFamily.AUTHORITY;
}
public async hasTransactionFinality(): Promise<boolean> {
const currentConsensusAlgorithmFamily = await this.getConsensusAlgorithmFamily();

return consensusHasTransactionFinality(currentConsensusAlgorithmFamily);
}
public async invokeContract(
req: InvokeContractV1Request,
): Promise<InvokeContractV1Response> {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import {
ICactusPluginOptions,
ConsensusAlgorithmFamily,
} from "@hyperledger/cactus-core-api";

import { consensusHasTransactionFinality } from "@hyperledger/cactus-core";
import {
Checks,
Logger,
Expand Down Expand Up @@ -95,6 +95,11 @@ export class PluginLedgerConnectorCorda
> {
return ConsensusAlgorithmFamily.AUTHORITY;
}
public async hasTransactionFinality(): Promise<boolean> {
const currentConsensusAlgorithmFamily = await this.getConsensusAlgorithmFamily();

return consensusHasTransactionFinality(currentConsensusAlgorithmFamily);
}

public getAspect(): PluginAspect {
return PluginAspect.LEDGER_CONNECTOR;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,10 @@ import {
ICactusPluginOptions,
} from "@hyperledger/cactus-core-api";

import { PluginRegistry } from "@hyperledger/cactus-core";
import {
consensusHasTransactionFinality,
PluginRegistry,
} from "@hyperledger/cactus-core";

import {
Logger,
Expand Down Expand Up @@ -202,7 +205,11 @@ export class PluginLedgerConnectorFabric
> {
return ConsensusAlgorithmFamily.AUTHORITY;
}
public async hasTransactionFinality(): Promise<boolean> {
const currentConsensusAlgorithmFamily = await this.getConsensusAlgorithmFamily();

return consensusHasTransactionFinality(currentConsensusAlgorithmFamily);
}
private async sshExec(
cmd: string,
label: string,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,10 @@ import {
IPluginKeychain,
} from "@hyperledger/cactus-core-api";

import { PluginRegistry } from "@hyperledger/cactus-core";
import {
PluginRegistry,
consensusHasTransactionFinality,
} from "@hyperledger/cactus-core";

import {
Checks,
Expand Down Expand Up @@ -205,7 +208,11 @@ export class PluginLedgerConnectorQuorum
> {
return ConsensusAlgorithmFamily.AUTHORITY;
}
public async hasTransactionFinality(): Promise<boolean> {
const currentConsensusAlgorithmFamily = await this.getConsensusAlgorithmFamily();

return consensusHasTransactionFinality(currentConsensusAlgorithmFamily);
}
public async invokeContract(
req: InvokeContractV1Request,
): Promise<InvokeContractV1Response> {
Expand Down

0 comments on commit a2d0582

Please sign in to comment.