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

Should be a drop-in replacement for IO::Socket::INET #3

Open
tadzik opened this issue Aug 24, 2014 · 2 comments
Open

Should be a drop-in replacement for IO::Socket::INET #3

tadzik opened this issue Aug 24, 2014 · 2 comments

Comments

@tadzik
Copy link

tadzik commented Aug 24, 2014

You should be able to replace IO::Socket::INET with IO::Socket::SSL and still have things working. So far it's missing some methods (like .get())

@ugexe
Copy link
Collaborator

ugexe commented Feb 10, 2015

@retupmoca ++ 75e66e6

Much of this was implemented in the commit above. There seems to only be a few details left, but .get was implemented https://github.com/ugexe/io-socket-ssl/commit/6402172924aa7b051baa91a516e9442a94bdebe8

@jjatria
Copy link
Contributor

jjatria commented Nov 9, 2020

It looks like one of the things missing before this can be closed is implementing the nl-in attribute. It looks like IO::Socket::SSL uses a single Str in input-line-separator that defaults to \n rather than the [ \n, \r\n ] in IO::Socket.

I've taken a stab at implementing this in a local branch. That includes some code to make it behave like IO::Socket when given a list of separators, including trying to match the longest token first. If this seems like something you might be interested in adding, then I can open a pull request and tidy up any lose ends (including tests and documentation changes).

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

3 participants