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

2024-01-25 Thread via GitHub 


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-unsubscr...@logging.apache.org

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

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

2024-01-25 Thread via GitHub 


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-unsubscr...@logging.apache.org

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

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

2024-01-25 Thread via GitHub 


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-unsubscr...@logging.apache.org

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

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

2024-01-25 Thread via GitHub 


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-unsubscr...@logging.apache.org.apache.org

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