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

[adminchannel] Topic mask has an undocumented/unintentional limit in number of characters #645

Closed
johnlage opened this issue Oct 21, 2014 · 1 comment

Comments

@johnlage
Copy link
Contributor

When i try to set a admittedly large topic mask, it gives me an error. The following is what i was trying to use as the mask at the time.

@tmask Current Topic: %s || Music: http://pastebin.com/wuL057E2 || Do not ping Kris if he's not on IRC. || Watch out for the poofing doctor || Halloween is coming! || We don't want a 卐 in here. || Woop, woop! Espernet got 4 new servers \o/
<Andromeda> sqlite3.OperationalError: near "s": syntax error (file "/var/lib/openshift/5425c25450044662d0000b79/app-root/runtime/repo/willie/willie/db.py", line 610, in update)

I think from what i gathered on IRC that this is unintentional.

Thanks,

John Lage

embolalia added a commit that referenced this issue Oct 27, 2014
There's still plenty of potential for problems with this. Stuff like
this is why #194 needs to happen. Related to #645.
@embolalia
Copy link
Contributor

The problem in Willie was not related to length, and is fixed. This mask will still have length issues (which should end up being handled by just truncating it), but that issue exists for all of IRC, and is well documented by the standard.

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