search

sous-chefs / selinux

Development repository for the selinux cookbook
https://supermarket.chef.io/cookbooks/selinux
Apache License 2.0
58 stars 70 forks source link

Correctly parse ports with multiple contexts #81

Closed detjensrobert closed 2 years ago

detjensrobert commented 2 years ago

Signed-off-by: Robert Detjens detjensrobert@osuosl.org

Description

If a port has multiple contexts assigned (e.g. adding httpd to 8080), selinux_port did not correctly recognize that and always assumed there was a single context, causing idempotency errors.

This fixes this and now handles multiple contexts correctly and was causing idempotency errors.

Issues Resolved

(none)

Check List

detjensrobert commented 2 years ago

CI is failing because the new macos runner doesnt have Vagrant installed anymore. Tests pass locally.

ramereth commented 2 years ago

CI is failing because the new macos runner doesnt have Vagrant installed anymore. Tests pass locally.

You need to change from using macos-latest to macos-10.15. They switched latest to point to 11 which doesn't have vagrant (yet).

kitchen-porter commented 2 years ago

Released as: 6.0.1