Skip to content

config::set should check for actual setting #16

@ju5t

Description

@ju5t

config::set checks if a setting exists in directadmin.conf. There are situations where this can be a problem. Especially when DirectAdmin changes some of their internal logic as happened with the SSL to ssl (upper to lowercase) transition.

We should use a similar approach as custombuild::config::set, but instead of checking the configuration, it should check the actual configuration like so:

./directadmin c | grep ^ssl=

and

./directadmin set ssl 1

This shouldn't be a major change.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions