cmseaton42 / node-ethernet-ip

A Lightweight Ethernet/IP API written to interface with Rockwell ControlLogix/CompactLogix Controllers.
MIT License
265 stars 106 forks source link

Port support #40

Open cmseaton42 opened 6 years ago

cmseaton42 commented 6 years ago

Description, Motivation, and Context

Adds support for use of additional tcp ports

How Has This Been Tested?

Still needs to be tested on a processor

Screenshots (if appropriate):

Types of changes

Checklist:

Related Issue

Fixes #38

cmseaton42 commented 6 years ago

@jhenson29 Can you check to make sure there are no adverse effects from this change when you get some time?

jhenson29 commented 6 years ago

Yes, I want to check with an actual router and using a different port. I should have time this week.

cmseaton42 commented 6 years ago

@jhenson29 Did you have a chance to check this in production?

jhenson29 commented 6 years ago

@cmseaton42 I haven’t. I’ve been drowning at work the last few weeks and haven’t really had time to work on this project. I have some hardware physically connected. I just need to make some time to configure the router and run the tests.

Sent with GitHawk