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

Bugfix/asymetric whitelist eprosima integration #3819

Merged

Conversation

rsanchez15
Copy link
Contributor

@rsanchez15 rsanchez15 commented Aug 29, 2023

See #3733 for more information on this PR

@rsanchez15 rsanchez15 marked this pull request as draft August 29, 2023 12:26
JesusPoderoso and others added 13 commits August 31, 2023 11:50
Signed-off-by: Juan López Fernández <[email protected]>
Signed-off-by: Juan Lopez Fernandez <[email protected]>
…peers and discovery server

Signed-off-by: Juan Lopez Fernandez <[email protected]>
Signed-off-by: Juan Lopez Fernandez <[email protected]>
Signed-off-by: Eduardo Ponz <[email protected]>
@rsanchez15 rsanchez15 force-pushed the bugfix/asymetric-whitelist-eprosima-integration branch from ab85367 to b660b6f Compare August 31, 2023 09:53
Signed-off-by: Raul Sanchez-Mateos <[email protected]>
@rsanchez15 rsanchez15 marked this pull request as ready for review August 31, 2023 09:58
@rsanchez15
Copy link
Contributor Author

@richiprosima please test this

@rsanchez15 rsanchez15 merged commit 962617b into eprosima/integration Aug 31, 2023
4 checks passed
@rsanchez15 rsanchez15 deleted the bugfix/asymetric-whitelist-eprosima-integration branch August 31, 2023 15:00
damien-projectx pushed a commit to figurerobotics/Fast-DDS that referenced this pull request Sep 6, 2023
* Refs #18854: Asymmetric whitelist regression test

Signed-off-by: JesusPoderoso <[email protected]>

* Refs #18854: Fix Windows build error

Signed-off-by: JesusPoderoso <[email protected]>

* Refs #18854: Apply rev suggestions

Signed-off-by: JesusPoderoso <[email protected]>

* Refs #19203: Add more test cases

Signed-off-by: Juan López Fernández <[email protected]>

* Refs #19203: Asymmetric whitelist matching fix: transform_remote_locators refactor

Signed-off-by: Juan López Fernández <[email protected]>

* Refs #19203: Tiny fixes

Signed-off-by: Juan Lopez Fernandez <[email protected]>

* Refs #19203: Add warnings for non-localhost local address in initial peers and discovery server

Signed-off-by: Juan Lopez Fernandez <[email protected]>

* Refs #19203: Bonus fix: TCPv6 + whitelist

Signed-off-by: Juan Lopez Fernandez <[email protected]>

* Refs #19203: Avoid API/ABI break

Signed-off-by: Juan Lopez Fernandez <[email protected]>

* Refs #19203: Fix TCP when no whitelist and initial peer != localhost

Signed-off-by: Eduardo Ponz <[email protected]>

* Refs #19203: Improve some comments

Signed-off-by: Eduardo Ponz <[email protected]>

* Refs #19203: Uncrustify

Signed-off-by: Eduardo Ponz <[email protected]>

* Refs #19203: Fix missing include

Signed-off-by: Eduardo Ponz <[email protected]>

* Rebase to eprosima/integration branch

Signed-off-by: Raul Sanchez-Mateos <[email protected]>

---------

Signed-off-by: JesusPoderoso <[email protected]>
Signed-off-by: Juan López Fernández <[email protected]>
Signed-off-by: Juan Lopez Fernandez <[email protected]>
Signed-off-by: Eduardo Ponz <[email protected]>
Signed-off-by: Raul Sanchez-Mateos <[email protected]>
Co-authored-by: JesusPoderoso <[email protected]>
Co-authored-by: Juan López Fernández <[email protected]>
Co-authored-by: Eduardo Ponz <[email protected]>
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.

4 participants