Skip to content

Conversation

Yoav11
Copy link
Contributor

@Yoav11 Yoav11 commented Jul 20, 2025

Description

this PR was created during EuroPython 2025 Sprint Weekend.

It builds on #5176 authored by @elafontaine


📚 Documentation preview 📚: https://cpython-previews--136850.org.readthedocs.build/

@python-cla-bot
Copy link

python-cla-bot bot commented Jul 20, 2025

All commit authors signed the Contributor License Agreement.

CLA signed

@Yoav11 Yoav11 changed the title gh-28879: Add Date header if missing in smtplib send_message bpo-28879: Add Date header if missing in smtplib send_message Jul 20, 2025
@StanFromIreland
Copy link
Member

Please change the title to use the GitHub issue number.

@Yoav11 Yoav11 changed the title bpo-28879: Add Date header if missing in smtplib send_message gh-73065: Add Date header if missing in smtplib send_message Jul 20, 2025
@Yoav11 Yoav11 marked this pull request as ready for review July 20, 2025 08:22
@Yoav11 Yoav11 requested a review from a team as a code owner July 20, 2025 08:22
@tomasr8 tomasr8 added the sprint label Jul 20, 2025
HOST, self.port, local_hostname='localhost',
timeout=support.LOOPBACK_TIMEOUT)
self.addCleanup(smtp.close)
self.assertEqual(smtp.send_message(msg), {})
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am moving these assertions further up so that I can extract the date from the message in order to build the expected string

@StanFromIreland
Copy link
Member

@Yoav11 A little suggestion, when you complete a suggestion, whether that be by implementing it or providing a reason why you are not going to, please mark the conversation as "Resolved," this keeps the PR more organized.

Copy link
Contributor

@ilovelinux ilovelinux left a comment

Choose a reason for hiding this comment

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

Consider mocking the date with a non-UTC time of which we know the timezone

@StanFromIreland
Copy link
Member

Please do not use the Update Branch button unless necessary (e.g. fixing conflicts, jogging the CI, or very old PRs) as it uses valuable resources. For more information see the devguide.

Copy link
Contributor

@ilovelinux ilovelinux left a comment

Choose a reason for hiding this comment

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

LGTM! Well done

@ilovelinux
Copy link
Contributor

@StanFromIreland the PR should be ready to be merged

@ilovelinux
Copy link
Contributor

@StanFromIreland bump 🙂

@StanFromIreland
Copy link
Member

The email experts have been requested, please be patient.

@ilovelinux
Copy link
Contributor

Thanks for the update 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants