Skip to content

feat: read welcome message from a variable - #207

Merged
pohmelie merged 4 commits into
aio-libs:masterfrom
rwakulszowa:feat/modifiable-welcome-message
Aug 4, 2026
Merged

feat: read welcome message from a variable#207
pohmelie merged 4 commits into
aio-libs:masterfrom
rwakulszowa:feat/modifiable-welcome-message

Conversation

@rwakulszowa

Copy link
Copy Markdown
Contributor

What do these changes do?

Allow setting a welcome message (aka banner) through a class variable. This is the message that the server replies with on the first connection.

Are there changes in behavior for the user?

Yes, but it's just a hardcoded string.

Checklist

  • I think the code is well written
  • Unit tests for the changes exist
  • Documentation reflects the changes

@rwakulszowa

Copy link
Copy Markdown
Contributor Author

Just a tiny thing that came up when I was using the library. It can be achieved by overriding the greeting method, but I figured I may not be the only person doing the exact same thing, so maybe it makes sense to have an easier way of doing this.

If you want, I can also add a CLI option.

For comparison, pyftpdlib does allow this:
https://github.com/giampaolo/pyftpdlib/blame/839ce0aa57d091873b56eaa657fca11e3dd23937/docs/cli.rst#L94

Comment thread src/aioftp/server.py Outdated
@rwakulszowa
rwakulszowa force-pushed the feat/modifiable-welcome-message branch 2 times, most recently from 3da80cf to a3456fc Compare August 3, 2026 15:04
@pohmelie

pohmelie commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

I will fix linter errors later today

@rwakulszowa rwakulszowa changed the title feat: read welcome message from a class variable feat: read welcome message from a variable Aug 4, 2026
Allow setting a welcome message (aka banner) through a variable.
This is the message that the server replies with on the first
connection.
@rwakulszowa
rwakulszowa force-pushed the feat/modifiable-welcome-message branch from a3456fc to 8c4283c Compare August 4, 2026 15:47
@codecov

codecov Bot commented Aug 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.94%. Comparing base (226ad64) to head (02de87b).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
- Coverage   97.94%   97.94%   -0.01%     
==========================================
  Files           6        6              
  Lines        2095     2094       -1     
==========================================
- Hits         2052     2051       -1     
  Misses         43       43              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pohmelie
pohmelie merged commit f2d0723 into aio-libs:master Aug 4, 2026
9 checks passed
@pohmelie

pohmelie commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Thank you for the contribution

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