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

Updating the certExpired option #53

Merged
merged 5 commits into from
Apr 14, 2012
Merged

Conversation

jonrohan
Copy link
Contributor

@jonrohan jonrohan commented Nov 1, 2011

No description provided.

@jonrohan
Copy link
Contributor Author

jonrohan commented Nov 1, 2011

also, sorry about the new pull request, I wasn't sure how to update the last one

@martynsmith
Copy link
Owner

Sorry for the delay, but I'm happy to merge this. Unfortunately you (like everyone else it seems) have patched the NAMES reply parsing code (because clearly it was exploding for lots of people).

Anyway, if you could fix up your patch to cleanly apply I'll happily merge it :-)

Thanks,

martynsmith added a commit that referenced this pull request Apr 14, 2012
Updating the certExpired option
@martynsmith martynsmith merged commit 3b3c01f into martynsmith:master Apr 14, 2012
Half-Shot referenced this pull request in matrix-org/node-irc Feb 23, 2019
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

Successfully merging this pull request may close these issues.

4 participants