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.

Sign up for GitHub

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

Jump to bottom

Add customizable transport: ignore participant flags, non-builtin descriptors and interface whitelist #42

Merged
merged 9 commits into from
Jun 15, 2023
Merged

Add customizable transport: ignore participant flags, non-builtin descriptors and interface whitelist #42

merged 9 commits into from
Jun 15, 2023

Conversation

juanlofer-eprosima
Copy link
Contributor

@juanlofer-eprosima juanlofer-eprosima commented May 29, 2023 •
edited
Loading

This was referenced May 29, 2023
Draft
Merged
@juanlofer-eprosima juanlofer-eprosima mentioned this pull request Jun 13, 2023
Merged
rsanchez15
rsanchez15 reviewed Jun 14, 2023
View reviewed changes
Copy link
Contributor

@rsanchez15 rsanchez15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing tests

ddspipe_yaml/include/ddspipe_yaml/yaml_configuration_tags.hpp Outdated Show resolved Hide resolved
@irenebm irenebm mentioned this pull request Jun 14, 2023
Merged
@rsanchez15 rsanchez15 mentioned this pull request Jun 15, 2023
Closed
4 tasks
rsanchez15 and others added 9 commits June 15, 2023 08:13
@rsanchez15 @juanlofer-eprosima
Add whitelist interfaces to DDS Pipe
6d25253 
Signed-off-by: Raul Sanchez-Mateos <raul@eprosima.com>
@rsanchez15 @juanlofer-eprosima
Add transport and ignore participant flags configuration
5d79e16 
Signed-off-by: Raul Sanchez-Mateos <raul@eprosima.com>
@juanlofer-eprosima
Disable builtin transport with UDP-only configuration
e60dec4 
Signed-off-by: Juan López Fernández <juanlopez@eprosima.com>
@juanlofer-eprosima
Apply whitelist configuration to WAN participants
f51eb2a 
Signed-off-by: Juan López Fernández <juanlopez@eprosima.com>
@juanlofer-eprosima
Uncrustify
bf734ce 
Signed-off-by: Juan López Fernández <juanlopez@eprosima.com>
@juanlofer-eprosima
Fix typo
c227a4e 
Signed-off-by: Juan López Fernández <juanlopez@eprosima.com>
@juanlofer-eprosima
Remove repeated push_back
65771ce 
Signed-off-by: Juan López Fernández <juanlopez@eprosima.com>
@juanlofer-eprosima
Make create_descriptor method public
59da404 
Signed-off-by: Juan López Fernández <juanlopez@eprosima.com>
@juanlofer-eprosima
Apply suggestions
d392c42 
Signed-off-by: Juan López Fernández <juanlopez@eprosima.com>
@rsanchez15 rsanchez15 merged commit 56367b8 into main Jun 15, 2023
@rsanchez15 rsanchez15 deleted the feature/customizable-transport branch June 15, 2023 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rsanchez15 rsanchez15 rsanchez15 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@juanlofer-eprosima @rsanchez15