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

Define MOTD_MAXSIZE in default_options.h #277

Closed
wants to merge 1 commit into from

Conversation

HansH111
Copy link
Contributor

add MOTD_MAXSIZE 2000 in default_options.h instead of hard coded value (80 * 25) in svr_chansession.c

This way you can increase the value in local_options.h if really needed.

@mkj
Copy link
Owner

mkj commented Feb 13, 2024

Is there really a good usecase for a motd larger than a minimal screen of text?

@HansH111
Copy link
Contributor Author

Some groups are using it to display information about the system and applications and the backup schedule.
Sometimes I get more than 3600 bytes depending on number applications running on a server.
I currently have more than 140 servers where the motd information is broken off and people are complaining...

Now I have to modify the source code to change the size. this way I can set a larger MOTD_MAXSIZE in localoptions.h without changing the sourcecode....

@mkj
Copy link
Owner

mkj commented Feb 14, 2024

Thanks. I've rebased it manually, github didn't want to in the PR.

@mkj mkj closed this Feb 14, 2024
@HansH111 HansH111 deleted the motd_maxsize branch February 21, 2024 13:11
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