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

Remove obsolete template options #2134

Merged
merged 1 commit into from
Jul 9, 2024
Merged

Conversation

jan-cerny
Copy link
Member

@jan-cerny jan-cerny commented Jul 9, 2024

The option --template allows users to provide either a remediation system URN or a custom XSLT template path. Instead of using the --template with URN users should use --fix-type. A custom XSLT path have never really worked. Users can apply custom XSLTs on their XML files using xsltsproc tool or other third party tools, they don't need oscap for that. We advise users to use --fix-type to specify the remediation type.

The --oval-template and --sce-template options are also useless because the result file names are defined by oscap itself, therefore there is no other sensible value for these options than the default value (hardcoded in oscap).

The option --template allows users to provide either a remediation
system URN or a custome XSLT template path. Instead of using the
--template with URN users should use --fix-type. A custom XSLT path have
never really worked. Users can apply custom XSLTs on their XML files
using xsltsproc tool or other third party tools, they don't need oscap
for that. We advise users to use --fix-type to specify the remediation
type.

The --oval-template and --sce-template options are also useless
because the result file names are defined by oscap itself, therefore
there is no other sensible value for these options than the default
value (hardcoded in oscap).
@jan-cerny jan-cerny added this to the 1.4 milestone Jul 9, 2024
@Mab879 Mab879 self-assigned this Jul 9, 2024
Copy link
Member

@Mab879 Mab879 left a comment

Choose a reason for hiding this comment

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

LGTM.

Thanks.

@Mab879 Mab879 merged commit 8f06d23 into OpenSCAP:main Jul 9, 2024
16 checks passed
comps added a commit to RHSecurityCompliance/contest that referenced this pull request Aug 18, 2024
It seems --template was removed in OpenSCAP v1.4, and --fix-type
seems to be a better fit here anyway.

    OpenSCAP/openscap#2134

It's also used in other ansible tests instead of --template.

The resulting playbook is exactly the same.

Signed-off-by: Jiri Jaburek <comps@nomail.dom>
mildas pushed a commit to RHSecurityCompliance/contest that referenced this pull request Aug 20, 2024
It seems --template was removed in OpenSCAP v1.4, and --fix-type
seems to be a better fit here anyway.

    OpenSCAP/openscap#2134

It's also used in other ansible tests instead of --template.

The resulting playbook is exactly the same.

Signed-off-by: Jiri Jaburek <comps@nomail.dom>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants