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

MODE change resulting in constant crash #171

Closed
katanacrimson opened this issue Jun 2, 2013 · 4 comments
Closed

MODE change resulting in constant crash #171

katanacrimson opened this issue Jun 2, 2013 · 4 comments

Comments

@katanacrimson
Copy link
Contributor

TypeError: Cannot call method 'indexOf' of undefined
    at .../karmic/node_modules/irc/lib/irc.js:233:54

Running 0.3.6. Seems the indexof check is hitting something that doesn't exist somewhere.

@katanacrimson
Copy link
Contributor Author

Another crash.

TypeError: Cannot call method 'replace' of undefined
    at .../karmic/node_modules/irc/lib/irc.js:239:71

Is this a case sensitivity problem at all or its kin?

@quentinxs
Copy link
Contributor

@damianb What's actually showing up as undefined when running the commands through the client, and what were the lines being sent? ...seeing as these look related, and we know it's not the whole line. The lines going through the library at the time of the crashes along with a little investigating to narrow the scope of the problem greatly helps towards diagnosing and fixing the underlying cause(s).

@katanacrimson
Copy link
Contributor Author

I've yet to insert additional debug lines at this point - I'm not actually sure what lines are the problem as I have multiple connections ongoing in my project and i'm not logging what the line is to stdout. Will do when I get a chance at least, but tbh domain support should be looked into for future updates.

@katanacrimson
Copy link
Contributor Author

Gut says this was an issue with channel modes, like the infamous q/o user mode bug that there's been a couple PRs for. Just going to close this to reduce noise.

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

2 participants