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.

Sign up for GitHub

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

Jump to bottom

Switch to not_valid_after_utc and not_valid_before_utc #5129

Merged
merged 2 commits into from
Mar 19, 2025
Merged

Switch to not_valid_after_utc and not_valid_before_utc #5129

merged 2 commits into from
Mar 19, 2025

Conversation

jtschladen
Copy link
Contributor

Cleans up these warnings:

CryptographyDeprecationWarning: Properties that return a naïve datetime object have been deprecated. Please switch to not_valid_before_utc. return cert.not_valid_before

CryptographyDeprecationWarning: Properties that return a naïve datetime object have been deprecated. Please switch to not_valid_after_utc. return cert.not_valid_after

@jtschladen jtschladen enabled auto-merge March 18, 2025 18:50
@jtschladen jtschladen merged commit 1ffeaad into Netflix:main Mar 19, 2025
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmcrawford45 jmcrawford45 jmcrawford45 approved these changes

@hosssha hosssha Awaiting requested review from hosssha hosssha is a code owner

@charhate charhate Awaiting requested review from charhate charhate is a code owner

@douglasc-nflx douglasc-nflx Awaiting requested review from douglasc-nflx douglasc-nflx is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jtschladen @jmcrawford45