Skip to content

Quantum: Refactor OpenSSL padding modeling - #19908

Closed
bdrodes wants to merge 3 commits into
github:mainfrom
bdrodes:openssl_padding_refactor
Closed

Quantum: Refactor OpenSSL padding modeling#19908
bdrodes wants to merge 3 commits into
github:mainfrom
bdrodes:openssl_padding_refactor

Conversation

@bdrodes

Copy link
Copy Markdown
Contributor

Updates how padding is modeled for openSSL.

CopilotAI review requested due to automatic review settings June 27, 2025 17:56
@bdrodes
bdrodes requested a review from a team as a code ownerJune 27, 2025 17:56

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@nicolaswillnicolaswill changed the title Openssl padding refactorQuantum: Refactor OpenSSL padding modelingJun 29, 2025

@nicolaswillnicolaswill left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

QL-for-QL alerts

@bdrodes

Copy link
Copy Markdown
ContributorAuthor

QL-for-QL alerts

Maybe I'm using the UI wrong, but I seem to only detect these by scrolling through the diffs and hope I don't miss one. I saw 3 and I corrected those. Hopefully I didn't miss any.

@bdrodes

Copy link
Copy Markdown
ContributorAuthor

Previously merged into the branch that was merged into main in this PR #19944

Closing this PR

@bdrodesbdrodes closed this Oct 2, 2025
@nicolaswill
nicolaswill deleted the openssl_padding_refactor branch October 2, 2025 17:35
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@bdrodes@nicolaswill