Skip to content

Commit

Permalink
CORE-17821 Removing unused crypto topic (#1353)
Browse files Browse the repository at this point in the history
Removing the `crypto.ops.flow` topic from the API package as it's unused now that `CryptoFlowOps` events are travelling over HTTP/RPC rather than the Kafka message bus.
  • Loading branch information
ben-millar authored Nov 23, 2023
1 parent 77ff03a commit 8d56003
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,6 @@ private Crypto() {

public static final String RPC_HSM_REGISTRATION_MESSAGE_TOPIC = "crypto.hsm.rpc.registration";
public static final String RPC_HSM_REGISTRATION_MESSAGE_RESPONSE_TOPIC = getRPCResponseTopic(RPC_HSM_REGISTRATION_MESSAGE_TOPIC);
public static final String FLOW_OPS_MESSAGE_TOPIC = "crypto.ops.flow";
public static final String RPC_OPS_MESSAGE_TOPIC = "crypto.ops.rpc";
public static final String RPC_OPS_MESSAGE_RESPONSE_TOPIC = getRPCResponseTopic(RPC_OPS_MESSAGE_TOPIC);
public static final String REKEY_MESSAGE_TOPIC = "crypto.key.rotation.ops";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,6 @@ topics:
producers:
- crypto
config:
CryptoOpsFlowTopic:
name: crypto.ops.flow
consumers:
- crypto
producers:
- flow
config:
CryptoOpsRpcTopic:
name: crypto.ops.rpc
consumers:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,5 @@ public class SchemasTests {
void cryptoTopicsShouldLookNiceInJavaApi() {
assertNotNull(Schemas.Crypto.RPC_HSM_REGISTRATION_MESSAGE_TOPIC);
assertNotNull(Schemas.Crypto.RPC_OPS_MESSAGE_TOPIC);
assertNotNull(Schemas.Crypto.FLOW_OPS_MESSAGE_TOPIC);
}
}
2 changes: 1 addition & 1 deletion gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ cordaProductVersion = 5.2.0
# NOTE: update this each time this module contains a breaking change
## NOTE: currently this is a top level revision, so all API versions will line up, but this could be moved to
## a per module property in which case module versions can change independently.
cordaApiRevision = 5
cordaApiRevision = 6

# Main
kotlinVersion = 1.8.21
Expand Down

0 comments on commit 8d56003

Please sign in to comment.