You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by "uhoefel (via GitHub)" <gi...@apache.org> on 2024/01/25 13:36:43 UTC

[I] `layout.template.json.resolver.EventResolver` contains non-exported type (logging-log4j2)

uhoefel opened a new issue, #2247:
URL: https://github.com/apache/logging-log4j2/issues/2247

   ## Description
   
   Currently, `org.apache.logging.log4j.layout.template.json` does not export `org.apache.logging.log4j.layout.template.json.util`. That makes it impossible to implement `org.apache.logging.log4j.layout.template.json.resolver.EventResolver` (which is in the exported packages) as the following two methods
   ```java
       /**
        * Resolves the given {@code value} using the provided {@link JsonWriter}.
        */
       void resolve(V value, JsonWriter jsonWriter);
   
       /**
        * Resolves the given {@code value} using the provided {@link JsonWriter}.
        *
        * @param succeedingEntry false, if this is the first element in a collection; true, otherwise
        */
       default void resolve(V value, JsonWriter jsonWriter, boolean succeedingEntry) {
           resolve(value, jsonWriter);
       }
   ``` 
   depend on `JsonWriter`, which is from `org.apache.logging.log4j.layout.template.json.util` and hence not available (if one uses JPMS).
   
   ## Configuration
   
   **Version:** 2.21.1
   
   **Operating system:** Windows 10
   
   **JDK:**
   openjdk version "21" 2023-09-19
   OpenJDK Runtime Environment (build 21+35-2513)
   OpenJDK 64-Bit Server VM (build 21+35-2513, mixed mode, sharing)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@logging.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] `layout.template.json.resolver.EventResolver` contains non-exported type (logging-log4j2)

Posted by "uhoefel (via GitHub)" <gi...@apache.org>.
uhoefel commented on issue #2247:
URL: https://github.com/apache/logging-log4j2/issues/2247#issuecomment-1910672874

   You are correct, seems like I had a lingering older version around. Sorry for the noise and thanks for the good work!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] `layout.template.json.resolver.EventResolver` contains non-exported type (logging-log4j2)

Posted by "ppkarwasz (via GitHub)" <gi...@apache.org>.
ppkarwasz commented on issue #2247:
URL: https://github.com/apache/logging-log4j2/issues/2247#issuecomment-1910494524

   @uhoefel,
   
   Did you check with the newest release (2.22.1)? This problem was reported as #1895 and should have been fixed in version 2.22.0.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] `layout.template.json.resolver.EventResolver` contains non-exported type (logging-log4j2)

Posted by "uhoefel (via GitHub)" <gi...@apache.org>.
uhoefel closed issue #2247: `layout.template.json.resolver.EventResolver` contains non-exported type
URL: https://github.com/apache/logging-log4j2/issues/2247


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org