DALnet / bahamut

The Bahamut IRC Daemon
http://www.dal.net/?page=Bahamut
Other
58 stars 42 forks source link

Allow tab characters with cmode +c #116

Closed fidsah closed 4 years ago

fidsah commented 5 years ago

Currently, messages containing the tab character(ASCII character 9) are prevented from being transmitted to a channel by the current check for ASCII characters less than 31. This patch is intended to preserve the intent of stopping control codes from being transmitted to a channel, while allowing messages containing the tab character to be transmitted.

rscs commented 5 years ago

Hi, @fidsah ... can you tell us more how this request came about? What's the proposed use case?

Tab is a well-documented ASCII control character. In fact, I can't find a single site that DOESN'T consider it a control character.

Examples:

github-actions[bot] commented 4 years ago

Stale pull request message

eaescob commented 4 years ago

Closing this PR as it hasn't had activity in quite some time. @fidsah if you want to re-open this, please create an issue to start discussion instead. Thanks!