Author: markt Date: Wed Sep 24 18:48:36 2014 New Revision: 1627393 URL: http://svn.apache.org/r1627393 Log: It appears production signing service needs a valid file extension so provide one.
Modified: tomcat/trunk/java/org/apache/tomcat/buildutil/SignCode.java Modified: tomcat/trunk/java/org/apache/tomcat/buildutil/SignCode.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/buildutil/SignCode.java?rev=1627393&r1=1627392&r2=1627393&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/tomcat/buildutil/SignCode.java (original) +++ tomcat/trunk/java/org/apache/tomcat/buildutil/SignCode.java Wed Sep 24 18:48:36 2014 @@ -162,7 +162,7 @@ public class SignCode extends Task { SOAPElement commaDelimitedFileNames = requestSigningRequest.addChildElement("commaDelimitedFileNames", NS); - commaDelimitedFileNames.addTextNode(getFileNames(filesToSign.size())); + commaDelimitedFileNames.addTextNode(getFileNames(filesToSign)); SOAPElement application = requestSigningRequest.addChildElement("application", NS); @@ -290,20 +290,34 @@ public class SignCode extends Task { /** * Signing service requires unique files names. Since files will be returned - * in order, use dummy names that we know are unique. + * in order, use dummy names that we know are unique but retain the file + * extension since the signing service appears to use it to figure out what + * to sign and how to sign it. */ - private static String getFileNames(int fileCount) { + private static String getFileNames(List<File> filesToSign) { StringBuilder sb = new StringBuilder(); boolean first = true; - for (int i = 0; i < fileCount; i++) { + for (int i = 0; i < filesToSign.size(); i++) { if (first) { first = false; } else { sb.append(','); } + File f = filesToSign.get(i); + String fileName = f.getName(); + int extIndex = fileName.lastIndexOf('.'); + String ext; + if (extIndex < 0) { + ext = null; + } else { + ext = fileName.substring(extIndex); + } sb.append(Integer.toString(i)); + if (ext != null) { + sb.append(ext); + } } return sb.toString(); } --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org