Skip to content

Bump max. line length#582

Draft
hloeung wants to merge 1 commit into42wim:masterfrom
hloeung:irc-bump-max-line-length
Draft

Bump max. line length#582
hloeung wants to merge 1 commit into42wim:masterfrom
hloeung:irc-bump-max-line-length

Conversation

@hloeung
Copy link
Copy Markdown
Collaborator

@hloeung hloeung commented Sep 9, 2024

Mattermost, and probably the others, allows for a longer max. line. In particular, topics (other messages are already using wordwrap).

Unfortunately, the upstream project, https://github.com/sorcix/irc, has been archived since Feb 21, 2023 and is currently in "read-only" mode.

@kot0dama
Copy link
Copy Markdown

It's possible some clients won't agree with a message exceeding the 510 (+CRLF) maximum length per the RFC, I would probably add an option to either enable longer lines, or even better to set the limit the user wants to allow.

@hloeung hloeung marked this pull request as draft September 10, 2024 01:43
@hloeung
Copy link
Copy Markdown
Collaborator Author

hloeung commented Sep 10, 2024

Sure, will make it configurable

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

Successfully merging this pull request may close these issues.

2 participants