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

Vote: Backport PR #19681 to 3.0 as a bug fix #60

Closed
romen opened this issue Nov 28, 2022 · 13 comments
Closed

Vote: Backport PR #19681 to 3.0 as a bug fix #60

romen opened this issue Nov 28, 2022 · 13 comments
Labels
accepted The policy change proposal was accepted by an OTC vote

Comments

@romen
Copy link
Member

romen commented Nov 28, 2022

Topic: Backport PR #19681 to 3.0 as a bug fix
Comment: Fundamentally OTC must decide if in the 3.0 release we should
         honor OSSL_PKEY_PARAM_EC_POINT_CONVERSION_FORMAT as set (and
         default to UNCOMPRESSED) in our provider implementations, and
         apply corresponding changes for handling legacy keys.
Proposed by: Nicola Tuveri
Issue link: https://github.com/openssl/technical-policies/issues/60
Public: yes
Opened: 2022-12-01
Closed: 2022-12-14
Accepted:  yes  (for: 4, against: 0, abstained: 5, not voted: 2)

  Dmitry     [+1]
  Matt       [ 0]
  Pauli      [+1]
  Tim        [ 0]
  Hugo       [+0]
  Richard    [  ]
  Shane      [ 0]
  Tomas      [ 0]
  Kurt       [  ]
  Matthias   [+1]
  Nicola     [+1]

Convenience links

@romen
Copy link
Member Author

romen commented Nov 28, 2022

For now this is still to be considered as a draft, and I haven't advertised it on the openssl-project ML.

@openssl/otc do you have feedback for the vote text before I formally open this vote?

@paulidale
Copy link
Contributor

I'm fine with the vote text.

@romen romen changed the title [DRAFT] Vote: Backport PR #19681 to 3.0 as a bug fix Vote: Backport PR #19681 to 3.0 as a bug fix Dec 1, 2022
@romen romen added the ready to vote The policy change proposal is ready to be voted on by the OTC label Dec 1, 2022
@romen
Copy link
Member Author

romen commented Dec 1, 2022

Opening the vote now

@paulidale
Copy link
Contributor

vote: +1

@t-j-h
Copy link
Member

t-j-h commented Dec 2, 2022

Vote: [0]

1 similar comment
@slontis
Copy link
Member

slontis commented Dec 2, 2022

Vote: [0]

@hlandau
Copy link
Member

hlandau commented Dec 2, 2022

Vote: [+0]

@mattcaswell
Copy link
Member

Vote: [0]

@t8m
Copy link
Member

t8m commented Dec 2, 2022

Vote: [0]

Unless there will be some minus votes, this will be probably a record passed vote in terms of the number of abstentions. 😁

@beldmit
Copy link
Member

beldmit commented Dec 13, 2022

vote: +1

@mspncp
Copy link
Contributor

mspncp commented Dec 13, 2022

Vote: [+1]

@romen
Copy link
Member Author

romen commented Dec 13, 2022

Closing the vote now as accepted:

Topic: Backport PR #19681 to 3.0 as a bug fix
Comment: Fundamentally OTC must decide if in the 3.0 release we should
         honor OSSL_PKEY_PARAM_EC_POINT_CONVERSION_FORMAT as set (and
         default to UNCOMPRESSED) in our provider implementations, and
         apply corresponding changes for handling legacy keys.
Proposed by: Nicola Tuveri
Issue link: https://github.com/openssl/technical-policies/issues/60
Public: yes
Opened: 2022-12-01
Closed: 2022-12-14
Accepted:  yes  (for: 4, against: 0, abstained: 5, not voted: 2)

  Dmitry     [+1]
  Matt       [ 0]
  Pauli      [+1]
  Tim        [ 0]
  Hugo       [+0]
  Richard    [  ]
  Shane      [ 0]
  Tomas      [ 0]
  Kurt       [  ]
  Matthias   [+1]
  Nicola     [+1]

@romen romen closed this as completed Dec 13, 2022
@romen romen added accepted The policy change proposal was accepted by an OTC vote and removed ready to vote The policy change proposal is ready to be voted on by the OTC labels Dec 13, 2022
@romen
Copy link
Member Author

romen commented Dec 14, 2022

See openssl/openssl#19901 for the 3.0-backport PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted The policy change proposal was accepted by an OTC vote
Projects
None yet
Development

No branches or pull requests

9 participants