Merge pull request #17 from apple/cb-remove-warnings

Lukasa · web-flow · commit 0cd2bb5ea535 · 2023-02-07T13:49:41.000Z
Silence the warnings I accidentally added
diff --git a/Sources/X509/Extension Types/BasicConstraints.swift b/Sources/X509/Extension Types/BasicConstraints.swift
@@ -109,7 +109,7 @@ struct BasicConstraintsValue: DERImplicitlyTaggable {
 
         // CA's must not assert the path len constraint field unless isCA is true.
         guard pathLenConstraint == nil || isCA else {
-            throw ASN1Error.invalidASN1Object(reason: "Invalid combination of isCA (\(isCA)) and path length constraint (\(pathLenConstraint)")
+            throw ASN1Error.invalidASN1Object(reason: "Invalid combination of isCA (\(isCA)) and path length constraint (\(String(describing: pathLenConstraint))")
         }
     }
 
diff --git a/Sources/X509/OCSP/OCSPResponse.swift b/Sources/X509/OCSP/OCSPResponse.swift
@@ -46,7 +46,7 @@ enum OCSPResponse: DERImplicitlyTaggable, Hashable {
                 guard let responseBytes,
                       responseBytes.responseType == .OCSP.basicResponse
                 else {
-                    throw ASN1Error.invalidASN1Object(reason: "Successful response does not have appropriate response bytes: \(responseBytes)")
+                    throw ASN1Error.invalidASN1Object(reason: "Successful response does not have appropriate response bytes: \(String(describing: responseBytes))")
                 }
                 return .successful(try BasicOCSPResponse(derEncoded: responseBytes.response.bytes))
             case .malformedRequest:

Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,7 @@ struct BasicConstraintsValue: DERImplicitlyTaggable {`
`109`	`109`
`110`	`110`	`// CA's must not assert the path len constraint field unless isCA is true.`
`111`	`111`	`guard pathLenConstraint == nil \|\| isCA else {`
`112`		`- throw ASN1Error.invalidASN1Object(reason: "Invalid combination of isCA (\(isCA)) and path length constraint (\(pathLenConstraint)")`
	`112`	`+ throw ASN1Error.invalidASN1Object(reason: "Invalid combination of isCA (\(isCA)) and path length constraint (\(String(describing: pathLenConstraint))")`
`113`	`113`	`}`
`114`	`114`	`}`
`115`	`115`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ enum OCSPResponse: DERImplicitlyTaggable, Hashable {`
`46`	`46`	`guard let responseBytes,`
`47`	`47`	`responseBytes.responseType == .OCSP.basicResponse`
`48`	`48`	`else {`
`49`		`- throw ASN1Error.invalidASN1Object(reason: "Successful response does not have appropriate response bytes: \(responseBytes)")`
	`49`	`+ throw ASN1Error.invalidASN1Object(reason: "Successful response does not have appropriate response bytes: \(String(describing: responseBytes))")`
`50`	`50`	`}`
`51`	`51`	`return .successful(try BasicOCSPResponse(derEncoded: responseBytes.response.bytes))`
`52`	`52`	`case .malformedRequest:`