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

Separate configuration and data ports #7

Open
petgit opened this issue Jan 15, 2015 · 4 comments
Open

Separate configuration and data ports #7

petgit opened this issue Jan 15, 2015 · 4 comments

Comments

@petgit
Copy link

petgit commented Jan 15, 2015

Now the data from the serial port and the configuration commands use the same port.
I think it is a good idea to separate this: i.e.
Telnet Port 23 fix for the AT commands.
And also possible for standard/warning/error output.
The os_sprintf data must recieved with an extra serial input at GPIO 2.

AT PORT set the port for the serial data (i.e 10001 as in the XPORT).
Then it would simpler to use also UDP , i.e.
AT PORT NR [TCP|UDP]

And also a client mode: The modul connects to a server or send UDP broadcasts.

@petgit petgit changed the title Change Setup and Data ports Separate configuration and data ports Jan 15, 2015
@monteslu
Copy link

+1

The current setup could be problematic if the bytes meant to be relayed to serial accidentally form a +++AT command

@fvpalha
Copy link

fvpalha commented May 29, 2015

Hi @petgit and @monteslu .

I changed the config tcp port - fvpalha/ESP8266-transparent-bridge
Please, can you test?

@beckdac
Copy link
Owner

beckdac commented Jun 8, 2015

Thanks @fvpalha, I see your pull request and I'll try to test it as soon as I get new modules in sometime in the next week. This is great!

@Cougar
Copy link
Contributor

Cougar commented Aug 1, 2015

This commit is full of different changes in addition to second TCP port. @fvpalha could you separate new feature from other changes? The current patch is hard to read read and understand but also can't be simply merged with upstream code.

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

No branches or pull requests

5 participants