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

Web Server: Address of Fast DDS discovery server does NOT accept special characters #400

Closed
royito55 opened this issue Jun 6, 2023 · 4 comments · Fixed by #420
Closed
Assignees
Labels
bug Something isn't working
Milestone

Comments

@royito55
Copy link

royito55 commented Jun 6, 2023

Hi,

I found an issue while attempting to communicate 2 create3's (with a RPi4) with another PC.

In order to connect to both robots from another PC, the discovery servers must be run on different IDs.

This means that the nodes running from one of the create3 must have the variable ROS_DISCOVERY_SERVER=;192.168.186.3:11811 (note the preceding ;). Refer to DDS docs here and to my discussion with @EduPonz here.

However, when I try to add this in the web server App config, I get the message "Please set a valid ipv4 address and port for the discovery server."
envvar

Can this be changed so special characters are allowed in that field?

Thank you

@EduPonz
Copy link

EduPonz commented Jun 7, 2023

Thanks @royito55! @alsora could you please point to the correct person to address this?

@alsora
Copy link
Collaborator

alsora commented Jun 7, 2023

I don't have a lot of context on the discovery server, but this seems a fairly straight-forward change.
We have a new firmware that will be released soon and we can make sure that it will include this change.

@EduPonz
Copy link

EduPonz commented Jun 7, 2023

Basically, adding ; as an allowed character there will allow for:

  1. Connecting the create3 to several Discovery Servers
  2. Connecting the create3 to Discovery Servers with ID other than 0

My guess is that is a simple change on the input validation, but I cannot be sure of course.

@alsora
Copy link
Collaborator

alsora commented Jun 7, 2023

Yes, there's a regex in there and I just added the ; character.

I'll let you know as soon as the new firmware is publicly available.

@shamlian shamlian self-assigned this Jun 23, 2023
@shamlian shamlian added this to the G.5/H.2 milestone Jun 23, 2023
@shamlian shamlian assigned alsora and unassigned shamlian Jun 23, 2023
@shamlian shamlian added the bug Something isn't working label Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants