Skip to content

Commit

Permalink
fix: Multiple instances of Thunder Broker created during Ripple boots…
Browse files Browse the repository at this point in the history
…trap (#693)

* fix: Multiple instances of Thunder Broker created during Ripple bootstrap.
* fix: Addressed the review comment to compare the endpoint protocol field instead of key.
  • Loading branch information
Vinodsathyaseelan authored Dec 2, 2024
1 parent 1099087 commit b2a1d95
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions core/main/src/broker/endpoint_broker.rs
Original file line number Diff line number Diff line change
Expand Up @@ -458,6 +458,10 @@ impl EndpointBrokerState {

pub fn build_other_endpoints(&mut self, session: Option<AccountSession>) {
for (key, endpoint) in self.rule_engine.rules.endpoints.clone() {
// skip thunder endpoint as it is already built using build_thunder_endpoint
if let RuleEndpointProtocol::Thunder = endpoint.protocol {
continue;
}
let request = BrokerConnectRequest::new_with_sesssion(
key,
endpoint.clone(),
Expand Down

0 comments on commit b2a1d95

Please sign in to comment.