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

Pass the CRL down to CycloneDDS if it exists. #325

Merged
merged 1 commit into from
Jul 8, 2021

Conversation

clalancette
Copy link
Contributor

If the rmw_dds_common layer tells us that the CRL exists,
set the appropriate property so CycloneDDS will honor it.

Signed-off-by: Chris Lalancette clalancette@openrobotics.org

Note that this can be merged at any time, but will have no effect until ros2/rmw_dds_common#52 is merged.

If the rmw_dds_common layer tells us that the CRL exists,
set the appropriate property so CycloneDDS will honor it.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Copy link
Collaborator

@eboasson eboasson left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@clalancette
Copy link
Contributor Author

CI for this is in ros2/rmw_dds_common#52 (comment) . That was green, and this is approved, so I'm going to go ahead and merge this.

@clalancette clalancette merged commit ab3d209 into master Jul 8, 2021
@clalancette clalancette deleted the clalancette/add-crl branch July 8, 2021 13:15
clalancette added a commit to eboasson/rmw_cyclonedds that referenced this pull request May 18, 2022
If the rmw_dds_common layer tells us that the CRL exists,
set the appropriate property so CycloneDDS will honor it.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
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