Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

S3BucketPublisher: handle InterruptedException #92

Merged
merged 1 commit into from
Jul 19, 2016
Merged

S3BucketPublisher: handle InterruptedException #92

merged 1 commit into from
Jul 19, 2016

Commits on Jul 19, 2016

  1. S3BucketPublisher: handle InterruptedException 

    Previously, if the source file patterns given to the S3 publisher
matched no files, we tried to be nice and called validateAntFileMask(),
which gave a nicely formatted message in case some other pattern would
have matched.

However, if even validateAntFileMask() fails to match anything within
the bound limit, then we would get a big fat InterruptedException that
kills the whole publisher and marks the build as a failure.

This patch makes this consistent by just ignoring the exception if
nothing was found. IOW, if we have nothing to suggest to the user to
help them diagnose the issue, that shouldn't be a reason to die.
    @jlebon
    jlebon committed Jul 19, 2016
    Configuration menu
    Copy the full SHA
    9aebf41 View commit details
    Browse the repository at this point in the history