Skip to content

bugfix servers using robot_check=True+encoding=False#144

Merged
jquast merged 3 commits into
masterfrom
jq/bugfix-robot-check-and-encoding=False
May 23, 2026
Merged

bugfix servers using robot_check=True+encoding=False#144
jquast merged 3 commits into
masterfrom
jq/bugfix-robot-check-and-encoding=False

Conversation

@jquast

@jquast jquast commented May 23, 2026

Copy link
Copy Markdown
Owner
  • bugfix: servers using robot_check=True with encoding=False raised TypeError: buf expected bytes, got <class 'str'>. telnetlib3-server now also accepts --encoding=False
    CLI argument. latin1 encoding is used the default server shell and robot check.

* bugfix: servers using ``robot_check=True`` with ``encoding=False`` raised ``TypeError: buf
  expected bytes, got <class 'str'>``. ``telnetlib3-server`` now also accepts ``--encoding=False``
  CLI argument. ``latin1`` encoding is used the default server shell and robot check.
@codspeed-hq

codspeed-hq Bot commented May 23, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 18 untouched benchmarks


Comparing jq/bugfix-robot-check-and-encoding=False (f6aff71) with master (296cc45)

Open in CodSpeed

@codecov

codecov Bot commented May 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.79518% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 92.56%. Comparing base (296cc45) to head (f6aff71).

Files with missing lines Patch % Lines
telnetlib3/server_shell.py 98.14% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #144      +/-   ##
==========================================
+ Coverage   92.23%   92.56%   +0.32%     
==========================================
  Files          28       28              
  Lines        7100     7125      +25     
  Branches     1236     1245       +9     
==========================================
+ Hits         6549     6595      +46     
+ Misses        356      343      -13     
+ Partials      195      187       -8     

☔ View full report in Codecov by Sentry.
📢 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.

@jquast jquast merged commit ce3155f into master May 23, 2026
22 checks passed
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.

1 participant