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.

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

TransferRunner discards vital error message information #15

Open
pdowler opened this issue Jan 5, 2017 · 0 comments
Open

TransferRunner discards vital error message information #15

pdowler opened this issue Jan 5, 2017 · 0 comments

Comments

@pdowler
Copy link
Member

pdowler commented Jan 5, 2017

In the series of catch statements (line 290+) the error message string used to sendError is just the base required message and discards additional detail from the exception. For example, the AccessControlException message says which node triggered the permission failure, which is required when trying to delete or move a container tree and the lack of write permission is somewhere in the tree.

Probably most of these errors could have the message from the exception suitably appended.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant