[
https://issues.apache.org/jira/browse/SLING-7358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16315210#comment-16315210
]
ASF GitHub Bot commented on SLING-7358:
---------------------------------------
Buuhuu opened a new pull request #1: SLING-7358: FileDistributionPackageBuilder
temp directory fallback
URL: https://github.com/apache/sling-org-apache-sling-distribution-core/pull/1
fallback to jvm's default temp directory if not configured in
FileDistributionPackageBuilder
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> FileDistributionPackageBuilder fails with no temp directory configured
> ----------------------------------------------------------------------
>
> Key: SLING-7358
> URL: https://issues.apache.org/jira/browse/SLING-7358
> Project: Sling
> Issue Type: Bug
> Components: Content Distribution
> Affects Versions: Content Distribution Core 0.2.10
> Reporter: Dirk Rudolph
> Priority: Minor
>
> When no temp directory is configured for the package builder
> [FileDistributionPackageBuilder|https://github.com/apache/sling-org-apache-sling-distribution-core/blob/master/src/main/java/org/apache/sling/distribution/packaging/impl/FileDistributionPackageBuilder.java#L86]
> uses null to create a new temp file which, according to the java docs will
> use the default temp directory.
> On the other hand [reading the file
> internally|https://github.com/apache/sling-org-apache-sling-distribution-core/blob/master/src/main/java/org/apache/sling/distribution/packaging/impl/FileDistributionPackageBuilder.java#L160]
> uses the {{File}} constructor passing {{null}} as directory to it. This
> causes a non-existing file to be returned.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
