Skip to content

Comments

Remove ENGINE usage from crypto/dsa/*#28566

Closed
jogme wants to merge 1 commit intoopenssl:feature/engineremovalfrom
jogme:1614
Closed

Remove ENGINE usage from crypto/dsa/*#28566
jogme wants to merge 1 commit intoopenssl:feature/engineremovalfrom
jogme:1614

Conversation

@jogme
Copy link
Contributor

@jogme jogme commented Sep 16, 2025

Resolves: openssl/project#1614

Signed-off-by: Norbert Pocs [email protected]

Checklist
  • documentation is added or updated
  • tests are added or updated

@jogme jogme linked an issue Sep 16, 2025 that may be closed by this pull request
@github-actions github-actions bot added severity: fips change The pull request changes FIPS provider sources severity: ABI change This pull request contains ABI changes labels Sep 16, 2025
@levitte levitte added branch: master Applies to master branch approval: review pending This pull request needs review by a committer labels Sep 16, 2025
@nhorman nhorman force-pushed the feature/engineremoval branch from e107df0 to d1d5468 Compare September 17, 2025 18:37
@nhorman nhorman force-pushed the feature/engineremoval branch from 34c1245 to aad1c4e Compare September 17, 2025 20:44
@jogme jogme mentioned this pull request Sep 18, 2025
2 tasks
@jogme
Copy link
Contributor Author

jogme commented Sep 18, 2025

As agreed the functions were moved back and the PR was rebased to the current feature/engineremoval. PTAL

@github-actions github-actions bot removed the severity: ABI change This pull request contains ABI changes label Sep 18, 2025
@t8m t8m added triaged: feature The issue/pr requests/adds a feature branch: feature The issue or PR is relevant only to one of the feature branches. and removed branch: master Applies to master branch labels Sep 18, 2025
@jogme jogme mentioned this pull request Sep 19, 2025
2 tasks
@paulidale paulidale added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Sep 23, 2025
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Sep 24, 2025
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@nhorman
Copy link
Contributor

nhorman commented Sep 24, 2025

merged to feature branch, thank yuou

@nhorman nhorman closed this Sep 24, 2025
openssl-machine pushed a commit that referenced this pull request Sep 24, 2025
Resolves: openssl/project#1614

Signed-off-by: Norbert Pocs <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from #28566)
mbroz pushed a commit to mbroz/openssl that referenced this pull request Oct 3, 2025
Resolves: openssl/project#1614

Signed-off-by: Norbert Pocs <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from openssl#28566)
nhorman pushed a commit to nhorman/openssl that referenced this pull request Oct 3, 2025
Resolves: openssl/project#1614

Signed-off-by: Norbert Pocs <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from openssl#28566)
mbroz pushed a commit to mbroz/openssl that referenced this pull request Dec 1, 2025
Resolves: openssl/project#1614

Signed-off-by: Norbert Pocs <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from openssl#28566)
nhorman pushed a commit to nhorman/openssl that referenced this pull request Dec 1, 2025
Resolves: openssl/project#1614

Signed-off-by: Norbert Pocs <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from openssl#28566)
nhorman pushed a commit to nhorman/openssl that referenced this pull request Dec 1, 2025
Resolves: openssl/project#1614

Signed-off-by: Norbert Pocs <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from openssl#28566)
mbroz pushed a commit to mbroz/openssl that referenced this pull request Dec 4, 2025
Resolves: openssl/project#1614

Signed-off-by: Norbert Pocs <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from openssl#28566)
nhorman pushed a commit to nhorman/openssl that referenced this pull request Dec 4, 2025
Resolves: openssl/project#1614

Signed-off-by: Norbert Pocs <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from openssl#28566)
mbroz pushed a commit to mbroz/openssl that referenced this pull request Dec 4, 2025
Resolves: openssl/project#1614

Signed-off-by: Norbert Pocs <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
(Merged from openssl#28566)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: feature The issue or PR is relevant only to one of the feature branches. severity: fips change The pull request changes FIPS provider sources triaged: feature The issue/pr requests/adds a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove engines from the crypto/dsa directory

7 participants