Merge pull request #55 from 398ja/release/0.1.4

Add logging enhancements and documentation updates

Author: tcheeric

CHANGELOG.md

@@ -58,4 +58,9 @@
 
 ### Notes
 - Enable debug logs with `logging.level.xyz.tcheeric=DEBUG` to see detailed request and configuration traces.
+## 0.1.4 — 2025-09-13
+
+### Enhancements
+- Add default logging configuration for SLF4J Simple in phoenixd-rest (`simplelogger.properties`).
+- Document logging usage and configuration.
 

README.md

@@ -46,3 +46,4 @@ version and `latest`, allowing consumers to pull the most recent build without s
   - [Docker](docs/how-to/docker.md)
   - [Code coverage](docs/how-to/coverage.md)
   - [Release](docs/how-to/release.md)
+  - [Logging](docs/reference/logging.md)

docs/reference/logging.md

@@ -0,0 +1,57 @@
+# Logging
+
+phoenixd-rest and phoenixd-base use SLF4J for logging with Lombok's `@Slf4j`.
+This lets your application choose the logging backend (Logback, Log4j2, slf4j-simple, etc.).
+
+## Quick Start (slf4j-simple)
+
+If your app does not already have a logging backend, add:
+
+```xml
+<dependency>
+  <groupId>org.slf4j</groupId>
+  <artifactId>slf4j-simple</artifactId>
+  <version>${slf4j.version}</version>
+</dependency>
+```
+
+The module ships a `simplelogger.properties` that sets:
+
+- `org.slf4j.simpleLogger.defaultLogLevel=info`
+- `org.slf4j.simpleLogger.log.xyz.tcheeric=debug`
+- date/time and thread name output
+
+Override levels at runtime with system properties, for example:
+
+```bash
+JAVA_TOOL_OPTIONS="-Dorg.slf4j.simpleLogger.defaultLogLevel=info \
+  -Dorg.slf4j.simpleLogger.log.xyz.tcheeric=debug"
+```
+
+## Spring Boot (Logback)
+
+Add to `application.properties`:
+
+```
+logging.level.xyz.tcheeric=DEBUG
+```
+
+Or to `logback-spring.xml`:
+
+```xml
+<configuration>
+  <logger name="xyz.tcheeric" level="DEBUG"/>
+  <root level="INFO">
+    <appender-ref ref="CONSOLE"/>
+  </root>
+</configuration>
+```
+
+## What’s Logged
+
+- HTTP requests: method, URI, timeout, status; error responses include a truncated body.
+- Configuration resolution: whether values come from ENV, system properties, or `app.properties` (base module).
+- Request factory decisions: Lightning address vs BOLT11 detection.
+
+Sensitive values (e.g. Authorization) are redacted.
+

phoenixd-base/pom.xml

@@ -4,7 +4,7 @@
     <parent>
         <groupId>xyz.tcheeric</groupId>
         <artifactId>phoenixd-java</artifactId>
-        <version>0.1.3</version>
+        <version>0.1.4</version>
     </parent>
 
     <artifactId>phoenixd-base</artifactId>

phoenixd-mock/pom.xml

@@ -6,7 +6,7 @@
     <parent>
         <groupId>xyz.tcheeric</groupId>
         <artifactId>phoenixd-java</artifactId>
-        <version>0.1.3</version>
+        <version>0.1.4</version>
     </parent>
 
     <artifactId>phoenixd-mock</artifactId>

phoenixd-model/pom.xml

@@ -4,7 +4,7 @@
     <parent>
         <groupId>xyz.tcheeric</groupId>
         <artifactId>phoenixd-java</artifactId>
-        <version>0.1.3</version>
+        <version>0.1.4</version>
     </parent>
 
     <artifactId>phoenixd-model</artifactId>

phoenixd-rest/pom.xml

@@ -4,12 +4,12 @@
     <parent>
         <groupId>xyz.tcheeric</groupId>
         <artifactId>phoenixd-java</artifactId>
-        <version>0.1.3</version>
+        <version>0.1.4</version>
     </parent>
 
     <artifactId>phoenixd-rest</artifactId>
     <packaging>jar</packaging>
-    <version>0.1.3</version>
+    <version>0.1.4</version>
     <name>phoenixd-rest</name>
     <url>https://maven.apache.org</url>
 

phoenixd-rest/src/main/java/xyz/tcheeric/phoenixd/operation/impl/DeleteOperation.java

@@ -2,6 +2,7 @@
 
 import lombok.NonNull;
 import lombok.SneakyThrows;
+import lombok.extern.slf4j.Slf4j;
 import xyz.tcheeric.phoenixd.common.rest.Operation;
 import xyz.tcheeric.phoenixd.common.rest.Request;
 import xyz.tcheeric.phoenixd.common.rest.util.Constants;
@@ -11,6 +12,7 @@
 import java.net.http.HttpRequest;
 import java.net.http.HttpRequest.Builder;
 
+@Slf4j
 public class DeleteOperation extends AbstractOperation {
 
     public DeleteOperation(HttpRequest httpRequest) {
@@ -20,11 +22,13 @@ public DeleteOperation(HttpRequest httpRequest) {
     @SneakyThrows
     public DeleteOperation(@NonNull String path) {
         super(Constants.HTTP_DELETE_METHOD, path, null);
+        if (log.isDebugEnabled()) log.debug("Initialized DELETE operation for path={}", path);
     }
 
     @SneakyThrows
     public DeleteOperation(@NonNull String path, @NonNull Request.Param requestParam) {
         super(Constants.HTTP_DELETE_METHOD, path, requestParam, null);
+        if (log.isDebugEnabled()) log.debug("Initialized DELETE operation for path={} with params", path);
     }
 
     @Override
@@ -40,11 +44,12 @@ public Operation removeHeader(@NonNull String key) {
         });
 
         this.httpRequest = requestBuilder.build();
+        if (log.isDebugEnabled()) log.debug("Removed header '{}' from DELETE request {} {}", key, httpRequest.method(), httpRequest.uri());
         return this;
     }
 
     @Override
     public Operation addHeader(String key, String value) {
         throw new UnsupportedOperationException("Not supported yet.");
     }
-}
+}

phoenixd-rest/src/main/java/xyz/tcheeric/phoenixd/operation/impl/GetOperation.java

@@ -2,12 +2,14 @@
 
 import lombok.NonNull;
 import lombok.SneakyThrows;
+import lombok.extern.slf4j.Slf4j;
 import xyz.tcheeric.phoenixd.common.rest.Request;
 import xyz.tcheeric.phoenixd.common.rest.util.Constants;
 import xyz.tcheeric.phoenixd.operation.AbstractOperation;
 
 import java.net.http.HttpRequest;
 
+@Slf4j
 public class GetOperation extends AbstractOperation {
 
     public GetOperation(HttpRequest httpRequest) {
@@ -17,10 +19,12 @@ public GetOperation(HttpRequest httpRequest) {
     @SneakyThrows
     public GetOperation(@NonNull String path) {
         super(Constants.HTTP_GET_METHOD, path, null);
+        if (log.isDebugEnabled()) log.debug("Initialized GET operation for path={}", path);
     }
 
     @SneakyThrows
     public GetOperation(@NonNull String path, @NonNull Request.Param requestParam) {
         super(Constants.HTTP_GET_METHOD, path, requestParam, null);
+        if (log.isDebugEnabled()) log.debug("Initialized GET operation for path={} with params", path);
     }
 }

phoenixd-rest/src/main/java/xyz/tcheeric/phoenixd/operation/impl/PatchOperation.java

@@ -2,21 +2,25 @@
 
 import lombok.NonNull;
 import lombok.SneakyThrows;
+import lombok.extern.slf4j.Slf4j;
 import xyz.tcheeric.phoenixd.common.rest.Operation;
 import xyz.tcheeric.phoenixd.common.rest.Request;
 import xyz.tcheeric.phoenixd.common.rest.util.Constants;
 import xyz.tcheeric.phoenixd.operation.AbstractOperation;
 
+@Slf4j
 public class PatchOperation extends AbstractOperation {
 
     @SneakyThrows
     public PatchOperation(@NonNull String path) {
         super(Constants.HTTP_PATCH_METHOD, path, null);
+        if (log.isDebugEnabled()) log.debug("Initialized PATCH operation for path={}", path);
     }
 
     @SneakyThrows
     public PatchOperation(@NonNull String path, @NonNull Request.Param param) {
         super(Constants.HTTP_PATCH_METHOD, path, param, null);
+        if (log.isDebugEnabled()) log.debug("Initialized PATCH operation for path={} with params", path);
     }
 
     @Override
@@ -26,6 +30,7 @@ public Operation removeHeader(String key) {
 
     @Override
     public Operation addHeader(String key, String value) {
+        if (log.isDebugEnabled()) log.debug("Adding header on PATCH operation: {}=<redacted? {}>", key, key.equalsIgnoreCase("Authorization") ? "yes" : "no");
         return super.addHeader(key, value);
     }