Skip to content

Add Call::refer#58

Merged
kbalt merged 2 commits intomainfrom
refer
Mar 3, 2026
Merged

Add Call::refer#58
kbalt merged 2 commits intomainfrom
refer

Conversation

@kbalt
Copy link
Copy Markdown
Owner

@kbalt kbalt commented Jan 18, 2026

No description provided.

@kbalt kbalt mentioned this pull request Jan 18, 2026
@kbalt kbalt force-pushed the refer branch 2 times, most recently from 4614e29 to 81bc8a2 Compare January 18, 2026 22:51
@hkrutzer
Copy link
Copy Markdown

hkrutzer commented Feb 1, 2026

Apologies for the late reply @kbalt and thanks for this feature! I ran into some unrelated issues that I had to solve before I could test it. I would like to test it some more but the functionality works! I did have to make some local changes to make ReferError ReferEvent and EventSubscriptionReceiver public.

@hkrutzer
Copy link
Copy Markdown

Tested this out further and it is working well for me. This could be useful:

diff --git a/sip/sip-ua/src/lib.rs b/sip/sip-ua/src/lib.rs
index 916dfde8aa..9b494c81f6 100644
--- a/sip/sip-ua/src/lib.rs
+++ b/sip/sip-ua/src/lib.rs
@@ -28,7 +28,7 @@
 mod registration;
 mod subscription;

-pub use call::{Call, CallError, CallEvent};
+pub use call::{Call, CallError, CallEvent, ReferError};
 pub use inbound_call::{AcceptCallError, InboundCall, InboundCallFromInviteError, NoMedia};
 pub use media_backend::MediaBackend;
 #[cfg(feature = "rtc")]
@@ -36,4 +36,6 @@
     Codec, MediaEvent, RtcMediaBackend, RtcMediaBackendError, RtpReceiver, RtpSender,
 };
 pub use outbound_call::{MakeCallCompletionError, MakeCallError, OutboundCall};
+pub use refer::ReferEvent;
 pub use registration::{RegisterError, RegistrarConfig, Registration};
+pub use subscription::EventSubscriptionReceiver;

esp. the last two because I was unable to have the compiler infer let mut refer_receiver: Option<_> = None; to be let mut refer_receiver: Option<EventSubscriptionReceiver<ReferEvent>> = None;

@kbalt kbalt force-pushed the refer branch 2 times, most recently from 17ca78c to 2eaca96 Compare March 3, 2026 21:20
@kbalt kbalt marked this pull request as ready for review March 3, 2026 21:51
@kbalt kbalt merged commit ecf31e1 into main Mar 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants