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

Move validation constraints to log4j-plugins-api #2172

Closed

Commits on Jan 8, 2024

  1. Split validators between l-plugins-api and l-plugins

    Plugin annotations are employed by `log4j-docgen` to generate
    documentation.
    
    Since the tool is compiled using Java 8 bytecode, we need to split
    `log4j-plugins` into:
    
     * an API part that contains mostly annotations and interfaces,
     * an implementation part that can use all the improvements of Java 17.
    
    In this commit:
    
     * we move the constraint annotation into a new artifact,
     * we make the `value` property of the `@Constraint` annotation
       optional,
     * we introduce a `ConstraintValidatorFactory` that produces the
       appropriate `ConstraintValidator` for each annotation. This way
       we don't need to reference `ConstraintValidator` implementations in
       the constraint annotations.
    ppkarwasz committed Jan 8, 2024
    Configuration menu
    Copy the full SHA
    e6610e8 View commit details
    Browse the repository at this point in the history
  2. Add old 2.x plugin validation annotations

    The old `2.x` annotations might still be used by some old plugins.
    ppkarwasz committed Jan 8, 2024
    Configuration menu
    Copy the full SHA
    cfd9f23 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    bbe9928 View commit details
    Browse the repository at this point in the history

Commits on Jan 9, 2024

  1. Configuration menu
    Copy the full SHA
    c5c2b87 View commit details
    Browse the repository at this point in the history