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

Better validate ACL configuration #1187

Merged
merged 1 commit into from
Oct 1, 2024
Merged

Commits on Sep 12, 2024

  1. Better validate ACL configuration

    The ACL configuration may not be a boolean or just any string. It must
    be a valid XACML. While we can't easily parse the XACML and check its
    correctness, we can at least check if the ACL seems to be XML.
    
    The background for this is that with Opencast 14, `acl = "false"` was a
    valid configuration. This silently changed with Opencast 15(?) and
    caused Studio to upload ACL files with the contents `false", causing
    problems in later processing steps of Opencast. This patch makes this at
    least easier to spot.
    lkiesow committed Sep 12, 2024
    Configuration menu
    Copy the full SHA
    18c472f View commit details
    Browse the repository at this point in the history