CupsClient - use string for URI not hostname, change default from ipp…

… to ipps.
gmuth · Jan 31, 2025 · 2c3fada · 2c3fada
1 parent df99693
commit 2c3fada
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/main/kotlin/de/gmuth/ipp/client/CupsClient.kt b/src/main/kotlin/de/gmuth/ipp/client/CupsClient.kt
@@ -1,7 +1,7 @@
 package de.gmuth.ipp.client
 
 /**
- * Copyright (c) 2020-2024 Gerhard Muth
+ * Copyright (c) 2020-2025 Gerhard Muth
  */
 
 import de.gmuth.ipp.client.IppOperationException.ClientErrorNotFoundException
@@ -21,10 +21,11 @@ import java.util.logging.Logger.getLogger
 
 // https://www.cups.org/doc/spec-ipp.html
 class CupsClient(
-    val cupsUri: URI = URI.create("ipp://localhost"),
+    val cupsUri: URI = URI.create("ipps://localhost"),
     val ippClient: IppClient = IppClient()
 ) {
-    constructor(host: String = "localhost") : this(URI.create("ipp://$host"))
+    @JvmOverloads
+    constructor(printerUri: String = "ipps://localhost") : this(URI.create(printerUri))
 
     private val logger = getLogger(javaClass.name)
     val config: IppConfig by ippClient::config

diff --git a/src/test/kotlin/de/gmuth/ipp/client/CupsClientTests.kt b/src/test/kotlin/de/gmuth/ipp/client/CupsClientTests.kt
@@ -22,7 +22,7 @@ class CupsClientTests {
     @Test
     fun constructors() {
         CupsClient()
-        CupsClient("host")
+        CupsClient("ipp://host")
     }
 
     @Test