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

Introduce secondary (Sub) '1.organizationalUnitName' field for DN #598

Closed
wants to merge 1 commit into from
Closed

Introduce secondary (Sub) '1.organizationalUnitName' field for DN #598

wants to merge 1 commit into from

Conversation

TinCanTech
Copy link
Collaborator

Add a final layer of granularity to X509 Distinguished Name.
Only used if '--req-sub-ou=some-user-data' is specified.

To minimize the noise to the user by this new field, change the way
that OpenSSL is called to build a CA. Always use '-batch' mode to
build the CA certificate.

User visible change when building a CA:

  • Instead of being prompted for each individual DN field, now the
    user is presented with a read-out of how the fields are currently
    set. There is now only a single confirmation that all fields are
    correct.
  • If '--req-sub-ou' is not set then it is not displayed.

Closes: #462 - The original proposal and prototype code.

Signed-off-by: Richard T Bonhomme [email protected]

Add a *final* layer of granularity to X509 Distinguished Name.
Only used if '--req-sub-ou=some-user-data' is specified.

To minimize the noise to the user by this new field, change the way
that OpenSSL is called to build a CA. Always use '-batch' mode to
build the CA certificate.

User visible change when building a CA:
* Instead of being prompted for each individual DN field, now the
  user is presented with a read-out of how the fields are currently
  set. There is now only a single confirmation that all fields are
  correct.
* If '--req-sub-ou' is not set then it is not displayed.

Closes: #462 - The original proposal and prototype code.

Signed-off-by: Richard T Bonhomme <[email protected]>
@TinCanTech TinCanTech self-assigned this Jun 17, 2022
@TinCanTech TinCanTech added enhancement feedback welcome Full-Approval Merge is imminent Major Changes Changes between Major 3.X version numbers - X is Major labels Jun 17, 2022
@TinCanTech TinCanTech added this to the v3.1.1-RC1 milestone Jun 17, 2022
@TinCanTech
Copy link
Collaborator Author

Following approval and merge, I will add a test to unit testing.

@TinCanTech
Copy link
Collaborator Author

This is still not satisfactory .. but it is close.

@TinCanTech TinCanTech added development Possible changes and removed Full-Approval Merge is imminent Major Changes Changes between Major 3.X version numbers - X is Major labels Jun 17, 2022
TinCanTech added a commit to TinCanTech/easy-rsa that referenced this pull request Jun 20, 2022
Add a *final* layer of granularity to X509 Distinguished Name.
Only used if --req-serial="<PRINTABLE>user data" is specified.

To minimize the noise to the user by this new field, change the way
that OpenSSL is called to build a CA: Always use '-batch' mode.

User visible change when building a CA:
* Instead of being prompted for each individual DN field, now the
  user is presented with a read-out of how the fields are currently
  set. There is now only a single confirmation that all fields are
  correct.
* If '--req-serial' is not used then 'serialNumber' is not displayed.

PRINTABLE: a-z,A-Z,0-9, -+/=.,?:()

Closes: OpenVPN#462 - The original proposal and prototype code.
Closes: OpenVPN#598 - Supersedes: Introduce 1.organizationalUnitName
Closes: OpenVPN#600 - Bugfix: Remove unused 'name' definition from SSL conf.

Signed-off-by: Richard T Bonhomme <[email protected]>
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant