Skip to content

Commit 15df816

Browse files
committed
Panic on secondary derivation
1 parent 2133a3d commit 15df816

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

datadog-crashtracker/src/shared/configuration.rs

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,6 @@ impl CrashtrackerConfiguration {
8888
unix_socket_path: Option<String>,
8989
demangle_names: bool,
9090
) -> anyhow::Result<Self> {
91-
panic!("new");
9291
Self::new_with_secondary(
9392
additional_files,
9493
create_alt_stack,
@@ -143,10 +142,16 @@ impl CrashtrackerConfiguration {
143142

144143
// If no secondary endpoint is provided but we have a primary endpoint,
145144
// derive a default secondary endpoint
145+
// let secondary_endpoint = secondary_endpoint.or_else(|| {
146+
// endpoint
147+
// .as_ref()
148+
// .and_then(|primary| derive_default_secondary_endpoint(primary))
149+
// });
150+
146151
let secondary_endpoint = secondary_endpoint.or_else(|| {
147-
endpoint
148-
.as_ref()
149-
.and_then(|primary| derive_default_secondary_endpoint(primary))
152+
let derived = derive_default_secondary_endpoint(endpoint.as_ref().unwrap());
153+
panic!("{}", format!("new_with_secondary: {:?}", derived));
154+
endpoint.as_ref().and_then(|primary| derived)
150155
});
151156

152157
// Note: don't check the receiver socket upfront, since a configuration can be interned

0 commit comments

Comments
 (0)