[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-10-25 Thread Herve Boutemy (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16960269#comment-16960269
 ] 

Herve Boutemy commented on WAGON-541:
-

[~ch...@christopherschultz.net]yes, the communication is between a build tool 
and a repository manager, once the repository manager implements the new 
protocol details, build tools will require to support it to benefit from its 
content, or they will get same result as current protocol through http/2: no 
detailed message
this is why we'll need to not only implement the protocol in Maven but also 
document it to give a chance to other build tools to benefit from it

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-10-25 Thread Christopher Schultz (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16959748#comment-16959748
 ] 

Christopher Schultz commented on WAGON-541:
---

> I'd evaluate something more standardized like {{application/problem+json}} if 
> it makes sense before we roll our own error format.

 

The only problem with {{application/problem+json}} is that it's basically a 
standard envelope around completely made-up contents. Unless the endpoints 
agree on what should be inside the envelope, we're back to just echoing 
whatever the server sends without any kind of interpretation.

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-10-25 Thread Herve Boutemy (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16959501#comment-16959501
 ] 

Herve Boutemy commented on WAGON-541:
-

yes, better idea: we'll define that precisely later. For now:
- I'll merge the PR
- I'll start a documentation on Maven to repository manager protocol on the site
- then we'll discuss precisely the replacement for reason phrase

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-10-24 Thread Michael Osipov (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16959001#comment-16959001
 ] 

Michael Osipov commented on WAGON-541:
--

[~brianf], I'd concur that in 95%+ the reason phrase will not contain any 
better help that then status code itself.

[~hboutemy], I'd evaluate something more standardized like 
{{application/problem+json}} if it makes sense before we roll our own error 
format.

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-10-24 Thread Herve Boutemy (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16958978#comment-16958978
 ] 

Herve Boutemy commented on WAGON-541:
-

since reason phrase will disappear in the future given http/2, we'll need a 
replacement: let's find a replacement (a header?)
waiting for this replacement definition, its implementation by repository 
managers, then deployment, we should keep the old HTTP/1.1 Reason Phrase

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-10-24 Thread Brian E Fox (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16958976#comment-16958976
 ] 

Brian E Fox commented on WAGON-541:
---

"

The reason-phrase element exists for the sole purpose of providing a
 textual description associated with the numeric status code, mostly
 out of deference to earlier Internet application protocols that were
 more frequently used with interactive text clients. A client SHOULD
 ignore the reason-phrase content.

reason-phrase = *( HTAB / SP / VCHAR / obs-text )"

 

Maven is in fact often used effectively as a text client and so the spirit of 
what is being done here is aligned with that original desire. Until such time 
as everyone switches over to http/2 there's no good reason to intentionally 
swallow this data, it can only help the user better understand what is going on.

 

 

 

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-10-24 Thread Michael Osipov (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16958923#comment-16958923
 ] 

Michael Osipov commented on WAGON-541:
--

I am referring to the RFC 7230, plain simple. It clearly says the client (Maven 
here) should ignore and there is nothing wrong to do so. Please read section 
3.1.2. HTTP/2 completely dropped the phrase anyway.

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-10-24 Thread Brian E Fox (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16958912#comment-16958912
 ] 

Brian E Fox commented on WAGON-541:
---

[~michael-o] I can't tell who or what you are referring to here?

 

The Reason Phrase is meant for the humans, the RFC is saying "don't write code 
that tries to interpret the reason-phrase text". 

 

Nothing being requested here is trying to do counter to the spec, on the 
contrary, we're saying the HUMANS need this reason phrase, it's why it exists 
in the spec. Maven should not swallow or hide this, which it didn't do until 
recently.

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-10-24 Thread Michael Osipov (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16958691#comment-16958691
 ] 

Michael Osipov commented on WAGON-541:
--

Citing from RFC 7230, *2014-06*:

bq. A client SHOULD ignore the reason-phrase content.

So folks, you had five years to get used to it.

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-10-13 Thread Herve Boutemy (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16950328#comment-16950328
 ] 

Herve Boutemy commented on WAGON-541:
-

[~apluche] did you try Peter's PR? does it fix the issue in your scenario?

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-10-03 Thread Peter lynch (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16943858#comment-16943858
 ] 

Peter lynch commented on WAGON-541:
---

PR up https://github.com/apache/maven-wagon/pull/56

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-02-11 Thread Peter lynch (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16764891#comment-16764891
 ] 

Peter lynch commented on WAGON-541:
---

just an update - I have the main code changes in place for adjusted wagon 
exception messages, with wagon tests. I am currently in the process of manually 
testing integrations with Maven, wagon-maven-plugin, maven-site-plugin and any 
other significant wagon project user I can find to make sure that these changes 
do not make any project regress. After I am satisfied I submit the PR for review

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-02-04 Thread Michael Osipov (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16760215#comment-16760215
 ] 

Michael Osipov commented on WAGON-541:
--

Feel free to do so.

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-02-04 Thread Peter lynch (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16760210#comment-16760210
 ] 

Peter lynch commented on WAGON-541:
---

[~michael-o] a system property or anything that makes reason phrase optionally 
displayed is not ideal.

People were using the status code and reason phrase to better understand why a 
build fails. This can help resolve build failures and get on with the important 
stuff more quickly.

When a build fails, it is too late to retroactively go back and set a system 
property to determine why it failed in the past. Users can't reasonably be 
expected to pre-set a new system property to see reason phrases for future 
possible failures either. They can't with complete certainty in advance if any 
intermediary server, firewall, HTTP Proxy, or repository manager will be 
customizing the reason phrase, which may provide useful guidance for fixing a 
build failure. The persons running builds are often not in control of 
infrastructure between their build and the remote endpoint. They can't predict 
how it will fail or what reason phrase will be returned.

I propose the spirit of the original change as I understand it be enhanced. 
Lets clean up code and make exception messages more consistent and less 
redundant across the 3 major HTTP wagon implementations. Exception messages 
could become consistently formatted with URL, status code and reason phrase 
while at the same time removing message redundancy.

I would be happy to provide a PR for both cleaning up messages and providing 
contextual useful Wagon exception messages that can help solve build failures 
in the most efficient manner. It might take me this week to offer this PR up 
for review.

My view point comes from trying to help enterprise customers who have Maven 
build failures trying to better understand what went wrong.

[~michael-o] how does it sound?



> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-02-04 Thread Michael Osipov (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16760200#comment-16760200
 ] 

Michael Osipov commented on WAGON-541:
--

bq. It seems like the main thrust of your concern was originally that so much 
of the message was repetitive information. What if you instead simply truncated 
the output and only showed the first 80 chars of the phrase?

Yes, I think that 90% of the users will see the very same information as the 
status code provided in contrast to {{Requested item is quarantined}}. The 
problem with 80 chars is that it won't simply scale. Given that almost all 
standard phrases are less than 80 chars, we'd we back at the beginning of 
message duplication. Having {{Access denied to:  , 
ReasonPhrase: Forbidden}} does not add any value to the {{Access denied}}.

I am thinking of something like 

{code}

  

  my-server
  
false 
  

  

{code}

This may work, but I need to evaluate this. I'd be happy if someone could 
provide a PR for this.

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-02-04 Thread Brian Fox (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=1676#comment-1676
 ] 

Brian Fox commented on WAGON-541:
-

A system property isn't a great solution. For companies trying to roll out a 
repo framework that does provide useful information, requiring all systems to 
set that property is essentially a non-starter.

 

It seems like the main thrust of your concern was originally that so much of 
the message was repetitive information. What if you instead simply truncated 
the output and only showed the first 80 chars of the phrase? 

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2019-02-03 Thread Michael Osipov (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16759371#comment-16759371
 ] 

Michael Osipov commented on WAGON-541:
--

I am considering adding a system property to enable to display the reason 
phrase. For most cases, this piece of information is useless.

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2018-12-17 Thread Aurelie Pluche (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16723169#comment-16723169
 ] 

Aurelie Pluche commented on WAGON-541:
--

You can go ahead and close it out.

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Fix For: waiting-for-feedback
>
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2018-12-16 Thread Michael Osipov (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16722588#comment-16722588
 ] 

Michael Osipov commented on WAGON-541:
--

Is there anything we stil need to sort out or can we close this one?

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Fix For: waiting-for-feedback
>
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2018-12-04 Thread Aurelie Pluche (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16708994#comment-16708994
 ] 

Aurelie Pluche commented on WAGON-541:
--

The answer to the initial question I asked boils down to: "The reason phrase 
handling has been removed altogether."

Thank you for your help.

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2018-12-04 Thread Michael Osipov (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16708963#comment-16708963
 ] 

Michael Osipov commented on WAGON-541:
--

The reason phrase handling has been removed altogether.

Looking at your sample screenshots doesn't reveal any beneficial information, 
how does "Internal Error Occured" and two UUIDs add more value that the 400 
itself. Infact Those two UUIDs should rather be response headers or part of the 
body. Citing from RFC 7230, chapter 3.1.2: "The reason-phrase element exists 
for the sole purpose of providing a textual description associated with the 
numeric status code, mostly out of deference to earlier Internet application 
protocols that were more frequently used with interactive text clients.". I 
fail to see the textucal representation of the status code here.

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2018-12-03 Thread Aurelie Pluche (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16707614#comment-16707614
 ] 

Aurelie Pluche commented on WAGON-541:
--

Thank you for the feedback - to answer your question: no, there is no deviation 
from the current RFCs. I see the reasoning behind removing the reason phrase 
for the 405 error code. Unfortunately, this isn't the only instance where the 
reasonPhrase isn't returned as expected.

I executed another call, this time using the Artifacts system where I know the 
reason phrase is being returned and the same thing occurs. The call 
intentionally returns a 400, which includes a reason phrase for why the call is 
failing. I used Fiddler to look at the returned headers and they are identical. 
I linked another set of images below. The first one is using maven 3.6.0 and 
the second 3.3.9. 

!MvnCmdLineV360V2.PNG!

!MvnCmdLineV339V2.PNG!

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV339V2.PNG, 
> MvnCmdLineV360.PNG, MvnCmdLineV360V2.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (WAGON-541) Command Line Not Showing ReasonPhrase for Errors

2018-12-03 Thread Michael Osipov (JIRA)


[ 
https://issues.apache.org/jira/browse/WAGON-541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16706971#comment-16706971
 ] 

Michael Osipov commented on WAGON-541:
--

This happened in WAGON-529, see the 
[commit|https://github.com/apache/maven-wagon/commit/53bdf1c2f1907868787fcdb2686e99eecbcc602a].
 "Method Not Allowed" gives no further benefit over the status code. Do you 
provide custom messages which deviate from the current RFCs?

> Command Line Not Showing ReasonPhrase for Errors
> 
>
> Key: WAGON-541
> URL: https://issues.apache.org/jira/browse/WAGON-541
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-http
>Affects Versions: 3.2.0
> Environment: Windows 10
>Reporter: Aurelie Pluche
>Priority: Minor
> Attachments: MvnCmdLineV339.PNG, MvnCmdLineV360.PNG
>
>
> Hi,
> I work in the Azure DevOps Artifacts Packaging team at Microsoft where we 
> provide a Maven service to our customers. We often use a Reason-Phrase to 
> return information on failed requests to customers. This functionality was 
> available in previous versions of maven but seems to have disappeared in 
> Maven 3.6.0. Was this intentional?
> I have included screenshots of the cmd line response using two different 
> maven versions (3.3.9 and 3.6.0). I intentionally made a call that would 
> return a 405 error to be able to get an error response. I also used the same 
> package.
> Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)