Skip to content
This repository has been archived by the owner on May 5, 2020. It is now read-only.

Split lines greater than 512 characters #54

Open
FugiTech opened this issue Oct 7, 2012 · 4 comments
Open

Split lines greater than 512 characters #54

FugiTech opened this issue Oct 7, 2012 · 4 comments

Comments

@FugiTech
Copy link
Contributor

FugiTech commented Oct 7, 2012

600 a's in a PRIVMSG just get sent out as is. Probably shouldn't happen.

@FugiTech
Copy link
Contributor Author

FugiTech commented Oct 8, 2012

Fixed for PRIVMSG and NOTICE. MODE needs fixing (eventually), anything else?

@ElementalAlchemist
Copy link
Contributor

When I get back to these, this needs a massive go-through of like every command in which input is sent back out.

@ElementalAlchemist
Copy link
Contributor

I just realized this is still here.
The new code isn't in this repo, but we should make sure to audit the new 0.4 code for this. I believe this is handled for everything 0.4 could send, but we should verify.

Maybe @Heufneutje can help verify.

@Heufneutje
Copy link

I believe this is handled as well. The only commands that should really be affected by it in 0.4 are PRIVMSG/NOTICE (already fixed by @ElementalAlchemist), considering we're already preventing insane config values that could make lines with other commands too long.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

3 participants