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

Udp interface does not work for non-localhost #580

Closed
onurturhan opened this Issue Sep 27, 2017 · 3 comments

Comments

Projects
None yet
4 participants
@onurturhan

onurturhan commented Sep 27, 2017

Udp interface works well until version 3.9.2 for "non-localhost" IP's. However, this interface doesn't send commands or receive telemetries from udp interface after version 4.0.0 or 4.0.1(tested version) for "non-localhost" IP's. (Tested on CentOS7, CentOS6, Windows7)

cmd_tlm_server.txt:
INTERFACE TEST_INT udp_interface.rb 10.1.233.20 1234 1235
TARGET TEST

@nickollasaranha

This comment has been minimized.

Show comment
Hide comment
@nickollasaranha

nickollasaranha Sep 28, 2017

It happens that UdpInterface has a:
@hostname = '127.0.0.1' if @hostname.casecmp('LOCALHOST')
and it should be
@hostname = '127.0.0.1' if @hostname.upcase.eql?('LOCALHOST')

nickollasaranha commented Sep 28, 2017

It happens that UdpInterface has a:
@hostname = '127.0.0.1' if @hostname.casecmp('LOCALHOST')
and it should be
@hostname = '127.0.0.1' if @hostname.upcase.eql?('LOCALHOST')

@jasonatball

This comment has been minimized.

Show comment
Hide comment
@jasonatball

jasonatball Sep 28, 2017

Collaborator

You are correct. I think this is my regression. I'll get it fixed ASAP.

Collaborator

jasonatball commented Sep 28, 2017

You are correct. I think this is my regression. I'll get it fixed ASAP.

@jasonatball jasonatball self-assigned this Sep 28, 2017

@jasonatball jasonatball closed this in #583 Sep 29, 2017

@ryanatball

This comment has been minimized.

Show comment
Hide comment
@ryanatball

ryanatball Sep 29, 2017

Member

This is now fixed in COSMOS 4.0.2

Member

ryanatball commented Sep 29, 2017

This is now fixed in COSMOS 4.0.2

@ryanatball ryanatball added the bug label Sep 29, 2017

@ryanatball ryanatball added this to the v4.0.2 milestone Sep 29, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment