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.

Sign up for GitHub

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

Jump to bottom

Improve data types #193

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Improve data types #193

wants to merge 6 commits into from

Commits on Aug 30, 2023

  1. Prefer Stdlib::Port to Integer for port number

    @smortex
    smortex committed Aug 30, 2023
    Configuration menu
    Copy the full SHA
    92f542e View commit details
    Browse the repository at this point in the history

  2. Set a lower bound on max number of jobs / vols 

    A negative value or 0 are not valid and should not be accepted by the
module.
    @smortex
    smortex committed Aug 30, 2023
    Configuration menu
    Copy the full SHA
    f4aed52 View commit details
    Browse the repository at this point in the history

  3. Prefer Stdlib::Absolutepath to String for filename

    @smortex
    smortex committed Aug 30, 2023
    Configuration menu
    Copy the full SHA
    40744ef View commit details
    Browse the repository at this point in the history

  4. Allow to use Sensitive to provide a password 

    In a future major version, we will remove the support for non-sensitive
passwords and stop providing a default value for them.
    @smortex
    smortex committed Aug 30, 2023
    Configuration menu
    Copy the full SHA
    03ed88e View commit details
    Browse the repository at this point in the history

  5. Rely on Bacula::Time where applicable

    @smortex
    smortex committed Aug 30, 2023
    Configuration menu
    Copy the full SHA
    ed08cdd View commit details
    Browse the repository at this point in the history

  6. Make String minimum length explicit 

    All user provided String are expected to be non-empty when they are not
undef.
    @smortex
    smortex committed Aug 30, 2023
    Configuration menu
    Copy the full SHA
    b9a9dbe View commit details
    Browse the repository at this point in the history