Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

[MSITE-705] Wagon logs displayed only if debug mode activated #95

Closed
wants to merge 1 commit into from
Closed

[MSITE-705] Wagon logs displayed only if debug mode activated #95

wants to merge 1 commit into from

Conversation

axel3rd
Copy link

@axel3rd axel3rd commented Oct 12, 2016

MSITE-705 : William Kemper patch + IT inclusion

@Tunaki
Copy link

Tunaki commented Oct 13, 2016

Is it really needed to deploy to a mock FTP server for the test? The Session: Disconnected logs, even when not in debug mode, can be reproduced with a simple push to a local file based repository. I can update the PR with this.

@axel3rd
Copy link
Author

axel3rd commented Oct 13, 2016

@Tunaki : It is to be closest to the original issue (even if this is not SFTP, a little bit more complex to mock). The FTP/SFTP logs are different => I try to found a common pattern for assert. But I agree, a push on local file is more simple & sustainable. I can change or provide a new PR ASAP based on this use case if considered as recevable for patch inclusion.

@Tunaki
Copy link

Tunaki commented Oct 13, 2016

I went ahead and updated the PR, will push this. Thanks!

@axel3rd
Copy link
Author

axel3rd commented Oct 13, 2016

OK, many thanks.

@axel3rd axel3rd closed this Oct 13, 2016
@axel3rd axel3rd deleted the MSITE-705 branch October 13, 2016 19:06
asfgit pushed a commit that referenced this pull request Oct 13, 2016
…sftp...

The debug listener on the Wagon instance is only added if we're running Maven in debug mode. This closes #95.

git-svn-id: https://svn.apache.org/repos/asf/maven/plugins/trunk@1764759 13f79535-47bb-0310-9956-ffa450edef68
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants