Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor interface & sdp server handling #50

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

SebaLukas
Copy link
Collaborator

Describe your changes

Added an interface check. If the interface is auto, the first found ipv6 interface will be selected. If no prober interface is provided, the tbd_controller throws an exception. The port from the sdp server is now reusable. The sdp socket binds now to the interface and joins multicast group

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • I read the contribution documentation and made sure that my changes meet its requirements

include/iso15118/detail/io/socket_helper.hpp Outdated Show resolved Hide resolved
src/iso15118/io/socket_helper.cpp Outdated Show resolved Hide resolved
…pv6 interface will be selected. If no prober interface is provided, the tbd_controller throws an exception. The port from the sdp server is now reusable. The sdp socket binds now to the interface and joins multicast group

Signed-off-by: Sebastian Lukas <sebastian.lukas@pionix.de>
Signed-off-by: Sebastian Lukas <sebastian.lukas@pionix.de>
Signed-off-by: Sebastian Lukas <sebastian.lukas@pionix.de>
@SebaLukas SebaLukas force-pushed the refactor/check-interface-and-sdp-server branch from df50777 to 882b571 Compare December 9, 2024 13:25
@SebaLukas SebaLukas requested a review from barsnick December 9, 2024 13:26
@SebaLukas SebaLukas marked this pull request as ready for review December 9, 2024 13:29
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