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

retryDelay not mentioned in the docs #446

Closed
trudheim opened this issue Feb 8, 2016 · 2 comments
Closed

retryDelay not mentioned in the docs #446

trudheim opened this issue Feb 8, 2016 · 2 comments

Comments

@trudheim
Copy link

trudheim commented Feb 8, 2016

Hi there,

The default retryDelay is set quite short (2000ms) and it can cause issues when connecting to some IRC servers that have flood and DoS protection. I needed to hunt in your code to find out what the actual option was to extend this delay. It would be nice if it was mentioned in the docs as others might run in to the same issue.

Kind regards,

/Anders

@creesch
Copy link

creesch commented Feb 9, 2016

you could edit the documentation yourself through this file https://github.com/martynsmith/node-irc/blob/master/docs/API.rst (don't be fooled by the github preview, if you show raw you'll see the documentation is actually in there.) and then do a pull request.

I am however not sure how active development is as I did a pull request myself a week ago and yet have to receive a response.

@jirwin
Copy link
Collaborator

jirwin commented Mar 27, 2016

I've added this to the docs. Thanks!

@jirwin jirwin closed this as completed Mar 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants