Merge pull request #204 from pmonks/issue-203

goneall · web-flow · commit b69b1674c789 · 2023-09-14T13:31:38.000-07:00
Addresses issue #203
diff --git a/src/main/java/org/spdx/library/model/SpdxModelFactory.java b/src/main/java/org/spdx/library/model/SpdxModelFactory.java
@@ -294,7 +294,7 @@ public static Optional<ModelObject> getModelObject(IModelStore modelStore, Strin
 			try {
 				return Optional.of(new ExternalSpdxElement(modelStore, documentUri, id, copyManager, true));
 			} catch(InvalidSPDXAnalysisException ex) {
-				logger.warn("Attempting to get a model object for an invalid SPDX ID.  Returning empty");
+				logger.warn("Attempting to get a model object for an invalid SPDX ID.  Returning empty", ex);
 				return Optional.empty();
 			}
 		}
diff --git a/src/main/java/org/spdx/library/model/license/LicenseExpressionParser.java b/src/main/java/org/spdx/library/model/license/LicenseExpressionParser.java
@@ -93,7 +93,7 @@ static AnyLicenseInfo parseLicenseExpression(String expression, IModelStore stor
 				// Add the expression to the error message to provide additional information to the user
 				throw new LicenseParserException(ex.getMessage()+" License expression: '"+expression+"'", ex);
 			} catch (EmptyStackException ex) {
-				throw new LicenseParserException("Invalid license expression: '"+expression+"' - check that every operator (e.g. AND and OR) has operators and that parenthesis are matched");
+				throw new LicenseParserException("Invalid license expression: '"+expression+"' - check that every operator (e.g. AND and OR) has operators and that parenthesis are matched", ex);
 			}
 		}
 	}
diff --git a/src/main/java/org/spdx/library/model/license/LicenseInfoFactory.java b/src/main/java/org/spdx/library/model/license/LicenseInfoFactory.java
@@ -87,7 +87,7 @@ public static AnyLicenseInfo parseSPDXLicenseString(String licenseString, @Nulla
 		} catch (LicenseParserException e) {
 			throw new InvalidLicenseStringException(e.getMessage(),e);
 		} catch (InvalidSPDXAnalysisException e) {
-			throw new InvalidLicenseStringException("Unexpected SPDX error parsing license string");
+			throw new InvalidLicenseStringException("Unexpected SPDX error parsing license string", e);
 		}
 	}
 
diff --git a/src/main/java/org/spdx/library/model/license/ListedLicenses.java b/src/main/java/org/spdx/library/model/license/ListedLicenses.java
@@ -85,13 +85,13 @@ private static Properties loadLicenseProperties() {
                 }
             } catch (IOException e) {
                 // Ignore it and fall through
-                logger.warn("IO Exception reading listed license properties file: " + e.getMessage());
+                logger.warn("IO Exception reading listed license properties file: " + e.getMessage(), e);
             } finally {
                 if (in != null) {
                     try {
                         in.close();
                     } catch (IOException e) {
-                        logger.warn("Unable to close listed license properties file: " + e.getMessage());
+                        logger.warn("Unable to close listed license properties file: " + e.getMessage(), e);
                     }
                 }
             }
@@ -108,16 +108,16 @@ private void initializeLicenseModelStore() {
         		try {
         			licenseModelStore = new SpdxListedLicenseWebStore();
         		} catch(InvalidSPDXAnalysisException ex) {
-        			logger.error("Unable to access the most current listed licenses from https://spdx.org/licenses - using locally cached licenses: "+ex.getMessage());
+        			logger.error("Unable to access the most current listed licenses from https://spdx.org/licenses - using locally cached licenses: "+ex.getMessage(), ex);
         			licenseModelStore = null;
         		}
         	}
         	if (licenseModelStore == null) {
         		try {
         			licenseModelStore = new SpdxListedLicenseLocalStore();
         		} catch(InvalidSPDXAnalysisException ex) {
-        			logger.error("Error loading cached SPDX licenses");
-        			throw new RuntimeException("Unexpected error loading SPDX Listed Licenses");
+        			logger.error("Error loading cached SPDX licenses", ex);
+        			throw new RuntimeException("Unexpected error loading SPDX Listed Licenses", ex);
         		}
         	}
         } finally {
diff --git a/src/main/java/org/spdx/library/model/license/SpdxListedLicense.java b/src/main/java/org/spdx/library/model/license/SpdxListedLicense.java
@@ -146,7 +146,7 @@ public String getLicenseTextHtml() throws InvalidLicenseTemplateException, Inval
 				try {
 					return SpdxLicenseTemplateHelper.templateTextToHtml(templateText);
 				} catch(LicenseTemplateRuleException ex) {
-					throw new InvalidLicenseTemplateException("Invalid license expression found in license text for license "+getName()+":"+ex.getMessage());
+					throw new InvalidLicenseTemplateException("Invalid license expression found in license text for license "+getName()+":"+ex.getMessage(), ex);
 				}
 			} else {
 				return SpdxLicenseTemplateHelper.formatEscapeHTML(this.getLicenseText());
@@ -179,7 +179,7 @@ public String getLicenseHeaderHtml() throws InvalidLicenseTemplateException, Inv
 				try {
 					return SpdxLicenseTemplateHelper.templateTextToHtml(templateText);
 				} catch(LicenseTemplateRuleException ex) {
-					throw new InvalidLicenseTemplateException("Invalid license expression found in standard license header for license "+getName()+":"+ex.getMessage());
+					throw new InvalidLicenseTemplateException("Invalid license expression found in standard license header for license "+getName()+":"+ex.getMessage(), ex);
 				}
 			} else {
 				return SpdxLicenseTemplateHelper.formatEscapeHTML(this.getStandardLicenseHeader());
diff --git a/src/main/java/org/spdx/library/referencetype/ListedReferenceTypes.java b/src/main/java/org/spdx/library/referencetype/ListedReferenceTypes.java
@@ -66,7 +66,7 @@ private ListedReferenceTypes() {
                 }
         	} catch (IOException e) {
                 logger.warn("IO Exception reading listed reference type properties file: " 
-                				+ e.getMessage() + ", loading properties from class properties file.");
+                				+ e.getMessage() + ", loading properties from class properties file.", e);
         	}
         	if (Objects.isNull(in)) {
         		try {
@@ -77,15 +77,15 @@ private ListedReferenceTypes() {
                 		logger.error("Unable to load listed reference type properties");
                 	}
                 } catch (IOException ex2) {
-                	logger.error("IO exception reading listed reference type properties from class properties file: "+ex2.getMessage());
+                	logger.error("IO exception reading listed reference type properties from class properties file: "+ex2.getMessage(), ex2);
                 }
         	}
         } finally {
             if (in != null) {
                 try {
                     in.close();
                 } catch (IOException e) {
-                    logger.warn("Unable to close listed license properties file: " + e.getMessage());
+                    logger.warn("Unable to close listed license properties file: " + e.getMessage(), e);
                 }
             }
         }
@@ -165,7 +165,7 @@ public URI getListedReferenceUri(String listedReferenceName) throws InvalidSPDXA
 			retval = new URI(SpdxConstants.SPDX_LISTED_REFERENCE_TYPES_PREFIX + listedReferenceName);
 		} catch (URISyntaxException e) {
 			logger.error("Error forming listed license URI",e);
-			throw new InvalidSPDXAnalysisException(listedReferenceName + " is not a valid SPDX listed reference type syntax.");
+			throw new InvalidSPDXAnalysisException(listedReferenceName + " is not a valid SPDX listed reference type syntax.",e);
 		}
     	if (!isListedReferenceType(retval)) {
     		throw new InvalidSPDXAnalysisException(listedReferenceName + " is not a valid SPDX listed reference type.");
diff --git a/src/main/java/org/spdx/storage/listedlicense/SpdxListedLicenseModelStore.java b/src/main/java/org/spdx/storage/listedlicense/SpdxListedLicenseModelStore.java
@@ -170,15 +170,15 @@ private void loadIds() throws InvalidSPDXAnalysisException {
                 ExceptionJsonTOC exceptionToc = gson.fromJson(tocJsonStr.toString(), ExceptionJsonTOC.class);
                 exceptionIds = exceptionToc.getExceptionIds();
             } catch (MalformedURLException e) {
-				throw new SpdxListedLicenseException("License TOC URL invalid") ;
+				throw new SpdxListedLicenseException("License TOC URL invalid", e) ;
 			} catch (IOException e) {
-				throw new SpdxListedLicenseException("I/O error reading license TOC");
+				throw new SpdxListedLicenseException("I/O error reading license TOC", e);
 			} finally {
             	if (reader != null) {
             		try {
 						reader.close();
 					} catch (IOException e) {
-						logger.warn("Unable to close JSON TOC reader");
+						logger.warn("Unable to close JSON TOC reader", e);
 					}
             	}
             }
@@ -326,23 +326,23 @@ private LicenseJson fetchLicenseJson(String idCaseInsensitive) throws InvalidSPD
 	                LicenseJson license = gson.fromJson(licenseJsonStr.toString(), LicenseJson.class);
 	                this.listedLicenseCache.put(id, license);
 	            } catch (MalformedURLException e) {
-					logger.error("Json license invalid for ID "+id);
-					throw new SpdxListedLicenseException("JSON license URL invalid for ID "+id);
+					logger.error("Json license invalid for ID "+id, e);
+					throw new SpdxListedLicenseException("JSON license URL invalid for ID "+id, e);
 				} catch (IOException e) {
-					logger.error("I/O error opening Json license URL");
-					throw new SpdxListedLicenseException("I/O Error reading license data for ID "+id);
+					logger.error("I/O error opening Json license URL", e);
+					throw new SpdxListedLicenseException("I/O Error reading license data for ID "+id, e);
 				} finally {
 	            	if (reader != null) {
 	            		try {
 							reader.close();
 						} catch (IOException e) {
-							logger.warn("Unable to close JSON TOC reader");
+							logger.warn("Unable to close JSON TOC reader", e);
 						}
 	            	} else if (jsonStream != null) {
 	            		try {
 							jsonStream.close();
 						} catch (IOException e) {
-							logger.warn("Unable to close JSON TOC input stream");
+							logger.warn("Unable to close JSON TOC input stream", e);
 						}
 	            	}
 				}
@@ -398,22 +398,22 @@ private ExceptionJson fetchExceptionJson(String idCaseInsensitive) throws Invali
 	                this.listedExceptionCache.put(id, exc);
 	            } catch (MalformedURLException e) {
 					logger.error("Json license invalid for ID "+id);
-					throw new SpdxListedLicenseException("JSON license URL invalid for ID "+id);
+					throw new SpdxListedLicenseException("JSON license URL invalid for ID "+id, e);
 				} catch (IOException e) {
 					logger.error("I/O error opening Json license URL");
-					throw new SpdxListedLicenseException("I/O Error reading license data for ID "+id);
+					throw new SpdxListedLicenseException("I/O Error reading license data for ID "+id, e);
 				} finally {
 	            	if (reader != null) {
 	            		try {
 							reader.close();
 						} catch (IOException e) {
-							logger.warn("Unable to close JSON TOC reader");
+							logger.warn("Unable to close JSON TOC reader", e);
 						}
 	            	} else if (jsonStream != null) {
 	            		try {
 							jsonStream.close();
 						} catch (IOException e) {
-							logger.warn("Unable to close JSON TOC input stream");
+							logger.warn("Unable to close JSON TOC input stream", e);
 						}
 	            	}
 				}
diff --git a/src/main/java/org/spdx/storage/listedlicense/SpdxListedLicenseWebStore.java b/src/main/java/org/spdx/storage/listedlicense/SpdxListedLicenseWebStore.java
@@ -63,7 +63,7 @@ private InputStream getUrlInputStream(URL url) throws IOException {
 			try {
 				redirectUrl = new URL(redirectUrlStr);
 			} catch(Exception ex) {
-				throw new IOException("Invalid redirect URL");
+				throw new IOException("Invalid redirect URL", ex);
 			}
 			if (!redirectUrl.getProtocol().toLowerCase().startsWith("http")) {
 				throw new IOException("Invalid redirect protocol");
diff --git a/src/main/java/org/spdx/storage/simple/StoredTypedItem.java b/src/main/java/org/spdx/storage/simple/StoredTypedItem.java
@@ -200,7 +200,7 @@ public boolean addValueToList(String propertyName, Object value) throws SpdxInva
 			}
 			return list.add(value);
 		} catch (Exception ex) {
-			throw new SpdxInvalidTypeException("Invalid list type for "+propertyName);
+			throw new SpdxInvalidTypeException("Invalid list type for "+propertyName, ex);
 		}
 	}
 	
@@ -222,7 +222,7 @@ public boolean removeTypedValueFromList(String propertyName, TypedValue value) t
 			}
 			return list.remove(value);
 		} catch (Exception ex) {
-			throw new SpdxInvalidTypeException("Invalid list type for "+propertyName);
+			throw new SpdxInvalidTypeException("Invalid list type for "+propertyName, ex);
 		}
 	}
 
@@ -257,7 +257,7 @@ public boolean removeValueFromList(String propertyName, Object value) throws Spd
 			}
 			return list.remove(value);
 		} catch (Exception ex) {
-			throw new SpdxInvalidTypeException("Invalid list type for "+propertyName);
+			throw new SpdxInvalidTypeException("Invalid list type for "+propertyName, ex);
 		}
 	}
 	
diff --git a/src/main/java/org/spdx/utility/compare/LicenseCompareHelper.java b/src/main/java/org/spdx/utility/compare/LicenseCompareHelper.java
@@ -244,14 +244,14 @@ public static String removeCommentChars(String s) {
 	            }
 	            return sb.toString();
 	        } catch (IOException e) {
-	            logger.warn("IO error reading strings?!?");
+	            logger.warn("IO error reading strings?!?", e);
 	            return s;
 	        } finally {
                 if (Objects.nonNull(reader)) {
                     try {
                         reader.close();
                     } catch (IOException e) {
-                        logger.warn("IO error closing a string reader?!?");
+                        logger.warn("IO error closing a string reader?!?", e);
                     }
                 }
 	        }
diff --git a/src/main/java/org/spdx/utility/compare/SpdxComparer.java b/src/main/java/org/spdx/utility/compare/SpdxComparer.java
@@ -901,7 +901,7 @@ private void compareDocumentContents() throws SpdxCompareException {
 				}
 			}
 		} catch(InvalidSPDXAnalysisException ex) {
-			throw new SpdxCompareException("Error getting SPDX document items: "+ex.getMessage());
+			throw new SpdxCompareException("Error getting SPDX document items: "+ex.getMessage(), ex);
 		}
 	}
 
diff --git a/src/main/java/org/spdx/utility/compare/SpdxSnippetComparer.java b/src/main/java/org/spdx/utility/compare/SpdxSnippetComparer.java
@@ -91,7 +91,7 @@ public void addDocumentSnippet(SpdxDocument spdxDocument,
 					this.differenceFound = true;
 				}
 			} catch (InvalidSPDXAnalysisException e) {
-				throw new SpdxCompareException("SPDX error getting byte range: "+e.getMessage());
+				throw new SpdxCompareException("SPDX error getting byte range: "+e.getMessage(), e);
 			}
 			try {
 			    Optional<StartEndPointer> lineRange = snippet.getLineRange();
@@ -111,22 +111,22 @@ public void addDocumentSnippet(SpdxDocument spdxDocument,
 					this.differenceFound = true;
 				}
 			} catch (InvalidSPDXAnalysisException e) {
-				throw new SpdxCompareException("SPDX error getting line range: "+e.getMessage());
+				throw new SpdxCompareException("SPDX error getting line range: "+e.getMessage(), e);
 			}
 			try {
 				SpdxFile fromFile = snippet.getSnippetFromFile();
 				SpdxFile fromFile2 = snippet2.getSnippetFromFile();
 				compareSnippetFromFiles(spdxDocument, fromFile, document2, fromFile2);
 			} catch (InvalidSPDXAnalysisException e) {
-				throw new SpdxCompareException("SPDX error getting snippet from file: "+e.getMessage());
+				throw new SpdxCompareException("SPDX error getting snippet from file: "+e.getMessage(), e);
 			}
 			try {
 				if (!SpdxComparer.stringsEqual(snippet2.getName(), snippet.getName())) {
 					this.nameEquals = false;
 					this.differenceFound = true;
 				}
 			} catch (InvalidSPDXAnalysisException e) {
-				throw new SpdxCompareException("SPDX error comparing snippet names: "+e.getMessage());
+				throw new SpdxCompareException("SPDX error comparing snippet names: "+e.getMessage(), e);
 			}
 		}
 		inProgress = false;

Original file line number	Diff line number	Diff line change
`@@ -294,7 +294,7 @@ public static Optional<ModelObject> getModelObject(IModelStore modelStore, Strin`
`294`	`294`	`try {`
`295`	`295`	`return Optional.of(new ExternalSpdxElement(modelStore, documentUri, id, copyManager, true));`
`296`	`296`	`} catch(InvalidSPDXAnalysisException ex) {`
`297`		`- logger.warn("Attempting to get a model object for an invalid SPDX ID. Returning empty");`
	`297`	`+ logger.warn("Attempting to get a model object for an invalid SPDX ID. Returning empty", ex);`
`298`	`298`	`return Optional.empty();`
`299`	`299`	`}`
`300`	`300`	`}`
Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ static AnyLicenseInfo parseLicenseExpression(String expression, IModelStore stor`
`93`	`93`	`// Add the expression to the error message to provide additional information to the user`
`94`	`94`	`throw new LicenseParserException(ex.getMessage()+" License expression: '"+expression+"'", ex);`
`95`	`95`	`} catch (EmptyStackException ex) {`
`96`		`- throw new LicenseParserException("Invalid license expression: '"+expression+"' - check that every operator (e.g. AND and OR) has operators and that parenthesis are matched");`
	`96`	`+ throw new LicenseParserException("Invalid license expression: '"+expression+"' - check that every operator (e.g. AND and OR) has operators and that parenthesis are matched", ex);`
`97`	`97`	`}`
`98`	`98`	`}`
`99`	`99`	`}`
Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ public static AnyLicenseInfo parseSPDXLicenseString(String licenseString, @Nulla`
`87`	`87`	`} catch (LicenseParserException e) {`
`88`	`88`	`throw new InvalidLicenseStringException(e.getMessage(),e);`
`89`	`89`	`} catch (InvalidSPDXAnalysisException e) {`
`90`		`- throw new InvalidLicenseStringException("Unexpected SPDX error parsing license string");`
	`90`	`+ throw new InvalidLicenseStringException("Unexpected SPDX error parsing license string", e);`
`91`	`91`	`}`
`92`	`92`	`}`
`93`	`93`
Original file line number	Diff line number	Diff line change
`@@ -85,13 +85,13 @@ private static Properties loadLicenseProperties() {`
`85`	`85`	`}`
`86`	`86`	`} catch (IOException e) {`
`87`	`87`	`// Ignore it and fall through`
`88`		`- logger.warn("IO Exception reading listed license properties file: " + e.getMessage());`
	`88`	`+ logger.warn("IO Exception reading listed license properties file: " + e.getMessage(), e);`
`89`	`89`	`} finally {`
`90`	`90`	`if (in != null) {`
`91`	`91`	`try {`
`92`	`92`	`in.close();`
`93`	`93`	`} catch (IOException e) {`
`94`		`- logger.warn("Unable to close listed license properties file: " + e.getMessage());`
	`94`	`+ logger.warn("Unable to close listed license properties file: " + e.getMessage(), e);`
`95`	`95`	`}`
`96`	`96`	`}`
`97`	`97`	`}`
`@@ -108,16 +108,16 @@ private void initializeLicenseModelStore() {`
`108`	`108`	`try {`
`109`	`109`	`licenseModelStore = new SpdxListedLicenseWebStore();`
`110`	`110`	`} catch(InvalidSPDXAnalysisException ex) {`
`111`		`- logger.error("Unable to access the most current listed licenses from https://spdx.org/licenses - using locally cached licenses: "+ex.getMessage());`
	`111`	`+ logger.error("Unable to access the most current listed licenses from https://spdx.org/licenses - using locally cached licenses: "+ex.getMessage(), ex);`
`112`	`112`	`licenseModelStore = null;`
`113`	`113`	`}`
`114`	`114`	`}`
`115`	`115`	`if (licenseModelStore == null) {`
`116`	`116`	`try {`
`117`	`117`	`licenseModelStore = new SpdxListedLicenseLocalStore();`
`118`	`118`	`} catch(InvalidSPDXAnalysisException ex) {`
`119`		`- logger.error("Error loading cached SPDX licenses");`
`120`		`- throw new RuntimeException("Unexpected error loading SPDX Listed Licenses");`
	`119`	`+ logger.error("Error loading cached SPDX licenses", ex);`
	`120`	`+ throw new RuntimeException("Unexpected error loading SPDX Listed Licenses", ex);`
`121`	`121`	`}`
`122`	`122`	`}`
`123`	`123`	`} finally {`
Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ private ListedReferenceTypes() {`
`66`	`66`	`}`
`67`	`67`	`} catch (IOException e) {`
`68`	`68`	`logger.warn("IO Exception reading listed reference type properties file: "`
`69`		`- + e.getMessage() + ", loading properties from class properties file.");`
	`69`	`+ + e.getMessage() + ", loading properties from class properties file.", e);`
`70`	`70`	`}`
`71`	`71`	`if (Objects.isNull(in)) {`
`72`	`72`	`try {`
`@@ -77,15 +77,15 @@ private ListedReferenceTypes() {`
`77`	`77`	`logger.error("Unable to load listed reference type properties");`
`78`	`78`	`}`
`79`	`79`	`} catch (IOException ex2) {`
`80`		`- logger.error("IO exception reading listed reference type properties from class properties file: "+ex2.getMessage());`
	`80`	`+ logger.error("IO exception reading listed reference type properties from class properties file: "+ex2.getMessage(), ex2);`
`81`	`81`	`}`
`82`	`82`	`}`
`83`	`83`	`} finally {`
`84`	`84`	`if (in != null) {`
`85`	`85`	`try {`
`86`	`86`	`in.close();`
`87`	`87`	`} catch (IOException e) {`
`88`		`- logger.warn("Unable to close listed license properties file: " + e.getMessage());`
	`88`	`+ logger.warn("Unable to close listed license properties file: " + e.getMessage(), e);`
`89`	`89`	`}`
`90`	`90`	`}`
`91`	`91`	`}`
`@@ -165,7 +165,7 @@ public URI getListedReferenceUri(String listedReferenceName) throws InvalidSPDXA`
`165`	`165`	`retval = new URI(SpdxConstants.SPDX_LISTED_REFERENCE_TYPES_PREFIX + listedReferenceName);`
`166`	`166`	`} catch (URISyntaxException e) {`
`167`	`167`	`logger.error("Error forming listed license URI",e);`
`168`		`- throw new InvalidSPDXAnalysisException(listedReferenceName + " is not a valid SPDX listed reference type syntax.");`
	`168`	`+ throw new InvalidSPDXAnalysisException(listedReferenceName + " is not a valid SPDX listed reference type syntax.",e);`
`169`	`169`	`}`
`170`	`170`	`if (!isListedReferenceType(retval)) {`
`171`	`171`	`throw new InvalidSPDXAnalysisException(listedReferenceName + " is not a valid SPDX listed reference type.");`
Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@ private InputStream getUrlInputStream(URL url) throws IOException {`
`63`	`63`	`try {`
`64`	`64`	`redirectUrl = new URL(redirectUrlStr);`
`65`	`65`	`} catch(Exception ex) {`
`66`		`- throw new IOException("Invalid redirect URL");`
	`66`	`+ throw new IOException("Invalid redirect URL", ex);`
`67`	`67`	`}`
`68`	`68`	`if (!redirectUrl.getProtocol().toLowerCase().startsWith("http")) {`
`69`	`69`	`throw new IOException("Invalid redirect protocol");`
Original file line number	Diff line number	Diff line change
`@@ -200,7 +200,7 @@ public boolean addValueToList(String propertyName, Object value) throws SpdxInva`
`200`	`200`	`}`
`201`	`201`	`return list.add(value);`
`202`	`202`	`} catch (Exception ex) {`
`203`		`- throw new SpdxInvalidTypeException("Invalid list type for "+propertyName);`
	`203`	`+ throw new SpdxInvalidTypeException("Invalid list type for "+propertyName, ex);`
`204`	`204`	`}`
`205`	`205`	`}`
`206`	`206`
`@@ -222,7 +222,7 @@ public boolean removeTypedValueFromList(String propertyName, TypedValue value) t`
`222`	`222`	`}`
`223`	`223`	`return list.remove(value);`
`224`	`224`	`} catch (Exception ex) {`
`225`		`- throw new SpdxInvalidTypeException("Invalid list type for "+propertyName);`
	`225`	`+ throw new SpdxInvalidTypeException("Invalid list type for "+propertyName, ex);`
`226`	`226`	`}`
`227`	`227`	`}`
`228`	`228`
`@@ -257,7 +257,7 @@ public boolean removeValueFromList(String propertyName, Object value) throws Spd`
`257`	`257`	`}`
`258`	`258`	`return list.remove(value);`
`259`	`259`	`} catch (Exception ex) {`
`260`		`- throw new SpdxInvalidTypeException("Invalid list type for "+propertyName);`
	`260`	`+ throw new SpdxInvalidTypeException("Invalid list type for "+propertyName, ex);`
`261`	`261`	`}`
`262`	`262`	`}`
`263`	`263`
Original file line number	Diff line number	Diff line change
`@@ -244,14 +244,14 @@ public static String removeCommentChars(String s) {`
`244`	`244`	`}`
`245`	`245`	`return sb.toString();`
`246`	`246`	`} catch (IOException e) {`
`247`		`- logger.warn("IO error reading strings?!?");`
	`247`	`+ logger.warn("IO error reading strings?!?", e);`
`248`	`248`	`return s;`
`249`	`249`	`} finally {`
`250`	`250`	`if (Objects.nonNull(reader)) {`
`251`	`251`	`try {`
`252`	`252`	`reader.close();`
`253`	`253`	`} catch (IOException e) {`
`254`		`- logger.warn("IO error closing a string reader?!?");`
	`254`	`+ logger.warn("IO error closing a string reader?!?", e);`
`255`	`255`	`}`
`256`	`256`	`}`
`257`	`257`	`}`