Skip to content

gh-136191: Unify commands to uppercase for wider support#136733

Open
MrExcitingDigits wants to merge 1 commit intopython:mainfrom
MrExcitingDigits:main
Open

gh-136191: Unify commands to uppercase for wider support#136733
MrExcitingDigits wants to merge 1 commit intopython:mainfrom
MrExcitingDigits:main

Conversation

@MrExcitingDigits
Copy link
Copy Markdown

@MrExcitingDigits MrExcitingDigits commented Jul 17, 2025

This decision to unify commands in uppercase comes from adherence to the industry standard and RFC 5321. It was posted in a python discussion and exists as issue 136191.

This decision to unify commands in uppercase comes from adherence to the industry standard and RFC 5321. It was posted in the python discussions here: https://discuss.python.org/t/unify-all-smtplib-commands-to-be-uppercase/97482
@MrExcitingDigits MrExcitingDigits requested a review from a team as a code owner July 17, 2025 03:00
@python-cla-bot
Copy link
Copy Markdown

The following commit authors need to sign the Contributor License Agreement:

CLA signed

@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented Jul 17, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

Comment thread Lib/smtplib.py
Comment on lines +40 to +41
# Upper case commands for wider support as enumerated in section 2.4 of RFC 5321
# by Steve Jacob <steve@mymangomail.com>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some change logs in a module's source code, but usually we don't update them nowadays. Instead, we mention them in the news entry (where the PR author's name can also be credited).

@github-actions
Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review stale Stale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants