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.

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

[3.9] bpo-43434: Move sqlite3.connect audit event to sqlite3.Connection.__init__ (GH-25818) #25822

Merged
merged 3 commits into from May 2, 2021

Conversation

erlend-aasland
Copy link
Contributor

@erlend-aasland erlend-aasland commented May 2, 2021

(cherry picked from commit c96cc08)

Co-authored-by: Erlend Egeberg Aasland erlend.aasland@innova.no

https://bugs.python.org/issue43434

@erlend-aasland
Copy link
Contributor Author

erlend-aasland commented May 2, 2021

@zooba I kept the tests. Let me know if I should remove them, as they were not present in 3.9.

@erlend-aasland
Copy link
Contributor Author

erlend-aasland commented May 2, 2021

@zooba Let me remove them; those PR's were not backported. Give me one minute.

UPDATE: Done!

…ion.__init__ (pythonGH-25818).

(cherry picked from commit c96cc08)

Co-authored-by: Erlend Egeberg Aasland <erlend.aasland@innova.no>
@zooba zooba changed the title [3.9] bpo-43434: Move sqlite3.connect audit event to sqlite3.Connection.__init__ [3.9] bpo-43434: Move sqlite3.connect audit event to sqlite3.Connection.__init__ (GH-25818) May 2, 2021
@erlend-aasland
Copy link
Contributor Author

There's a tiny spelling bug in the NEWS entry (were => was). I'll push a commit to fix it, if you don't mind.

@erlend-aasland
Copy link
Contributor Author

erlend-aasland commented May 2, 2021

I did the cherry-picking for 3.8 from the commit in main, iso. this one. Hope that's ok.

@erlend-aasland
Copy link
Contributor Author

BTW, I'm logging off for tonight. If there's an issue, I'll look at it first thing in the morning.

@zooba zooba merged commit ad73d16 into python:3.9 May 2, 2021
@miss-islington
Copy link
Contributor

Thanks @erlend-aasland for the PR, and @zooba for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry @erlend-aasland and @zooba, I had trouble checking out the 3.8 backport branch.
Please backport using cherry_picker on command line.
cherry_picker ad73d1657ca3591b1dd1d15bb80ce83ca7fc3223 3.8

@zooba zooba added needs backport to 3.8 only security fixes and removed needs backport to 3.8 only security fixes labels May 2, 2021
@miss-islington
Copy link
Contributor

Thanks @erlend-aasland for the PR, and @zooba for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@erlend-aasland erlend-aasland deleted the backport-c96cc08-3.9 branch May 2, 2021 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs backport to 3.8 only security fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants