vy commented on PR #3467:
URL: https://github.com/apache/logging-log4j2/pull/3467#issuecomment-2669414203

   Thanks so much Josh for bearing with us to deliver this feature. There are
   some last mile wrinkles to be ironed out in the tests, I will take care of
   them. (Please keep the PR open, I will continue working on it.)
   
   Op wo 19 feb 2025 om 17:59 schreef Josh Smith ***@***.***>
   
   > ***@***.**** commented on this pull request.
   > ------------------------------
   >
   > In
   > 
log4j-mongodb4/src/main/java/org/apache/logging/log4j/mongodb4/MongoDb4Connection.java
   > <https://github.com/apache/logging-log4j2/pull/3467#discussion_r1962053301>
   > :
   >
   > > @@ -67,21 +67,23 @@ public MongoDb4Connection(
   >              final ConnectionString connectionString,
   >              final MongoClient mongoClient,
   >              final MongoDatabase mongoDatabase,
   > +            final String collectionName,
   >
   > Trying the VERBOSE command above produced this
   >
   > ***@***.*** log4j-temp % MVNW_VERBOSE=true ./mvnw validate -N
   > Couldn't find MAVEN_HOME, downloading and installing it ...
   > Downloading from: 
https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.8/apache-maven-3.9.8-bin.zip
   > Downloading to: 
/var/folders/wm/96zyw_3x4cdbvycjl7btg8sc0000gn/T/tmp.4xLeCVRCI2/apache-maven-3.9.8-bin.zip
   > Found curl ... using curl
   >   % Total    % Received % Xferd  Average Speed   Time    Time     Time  
Current
   >                                  Dload  Upload   Total   Spent    Left  
Speed
   > 100 8979k  100 8979k    0     0  1584k      0  0:00:05  0:00:05 --:--:-- 
1402k
   > Error: Failed to validate Maven distribution SHA-256, your Maven 
distribution might be compromised.
   > If you updated your Maven version, you need to update the specified 
distributionSha256Sum property.
   > ***@***.*** log4j-temp %
   >
   > that's after doing a git reset to bring back the deleted
   > maven-wrapper.properties file.
   >
   > I suspect that this needs to be moved to an issue so we can separate the
   > resolution of this from the approval of the code changes for the appender.
   >
   > —
   > Reply to this email directly, view it on GitHub
   > 
<https://github.com/apache/logging-log4j2/pull/3467#discussion_r1962053301>,
   > or unsubscribe
   > 
<https://github.com/notifications/unsubscribe-auth/AAARTSMOWNLI4JFKYGYHUJ32QSZ7DAVCNFSM6AAAAABXF6KYPWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDMMRXGQ2DGMBRGU>
   > .
   > You are receiving this because you were mentioned.Message ID:
   > ***@***.***>
   >
   


-- 
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

Reply via email to