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

Deprecate MSFT_AdcsOcspExtension and replace with MSFT_AdcsAuthorityInformationAccess #101

Closed
PlagueHO opened this issue Sep 13, 2019 · 0 comments · Fixed by #104
Closed
Assignees
Labels
in progress The issue is being actively worked on by someone.

Comments

@PlagueHO
Copy link
Member

PlagueHO commented Sep 13, 2019

The MSFT_AdcsOcspExtension is implemented using an anti-pattern and also will potentially cause a conflict with another proposed resource - MSFT_AdcsAIAExtension - if implemented the way proposed. This is because both write into the same settings.

These resources should be refactored into a new combined resource
MSFT_AdcsAuthorityInformationAccess with a MOF like:

[ClassVersion("1.0.0"), FriendlyName("AdcsAuthorityInformationAccess")]
class MSFT_AdcsAuthorityInformationAccess: OMI_BaseResource
{
    [Key, Description("Specifies the resource is a single instance, the value must be 'Yes'."), ValueMap{"Yes"}, Values{"Yes"}] String IsSingleInstance;
    [Write, Description("Specifies the list of URIs that should be included in the AIA extension of the issued certificate.")] String AiaUri[];
    [Write, Description("Specifies the list of URIs that should be included in the Online Responder OCSP extension of the issued certificate")] String OCSPUri[];
    [Write, Description("Specifies if the service should be restarted.")] Boolean RestartService;
};
@PlagueHO PlagueHO changed the title Deprecate MSFT_AdcsOcspExtension Deprecate MSFT_AdcsOcspExtension and replace with MSFT_AdcsAuthorityInformationAccess Sep 13, 2019
@PlagueHO PlagueHO self-assigned this Sep 13, 2019
@PlagueHO PlagueHO added the in progress The issue is being actively worked on by someone. label Sep 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment