You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hop.apache.org by "roy20021 (via GitHub)" <gi...@apache.org> on 2023/11/17 09:09:46 UTC
[I] [Bug]: HttpPost / Authorization does not work (hop)
roy20021 opened a new issue, #3407:
URL: https://github.com/apache/hop/issues/3407
### Apache Hop version?
2.6.0
### Java version?
jdk-20.0.2.9-hotspot
### Operating system
Windows
### What happened?
The httppost transfomer is ignoring the "HTTP authentication" configuration:
but it sends:
Currently, I have worked around it by setting manually a "Authorization" header.
### Issue Priority
Priority: 2
### Issue Component
Component: Transforms
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Bug]: HttpPost / Authorization does not work (hop)
Posted by "roy20021 (via GitHub)" <gi...@apache.org>.
roy20021 closed issue #3407: [Bug]: HttpPost / Authorization does not work
URL: https://github.com/apache/hop/issues/3407
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Bug]: HttpPost / Authorization does not work (hop)
Posted by "hansva (via GitHub)" <gi...@apache.org>.
hansva commented on issue #3407:
URL: https://github.com/apache/hop/issues/3407#issuecomment-1816393414
Hi @roy20021 ,
You are right these transforms are turning a bit into a problem and have been created at separate points in time to solve very specific problems.
We also have request around supporting other forms of authentication (eg. Oauth, Bearer token, API key).
I think we should take a look at all the functionality these 3 transforms cover and see if we can combine all features in either one of them, or create a new transform.
The REST client seems like the best suspect to combine everything in, but I think that code might also need a refactoring to be a bit more readable.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Bug]: HttpPost / Authorization does not work (hop)
Posted by "nadment (via GitHub)" <gi...@apache.org>.
nadment commented on issue #3407:
URL: https://github.com/apache/hop/issues/3407#issuecomment-1819659386
For your information, deprecated transformations can be marked with the @Deprecated annotation to indicate this to users.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Bug]: HttpPost / Authorization does not work (hop)
Posted by "roy20021 (via GitHub)" <gi...@apache.org>.
roy20021 commented on issue #3407:
URL: https://github.com/apache/hop/issues/3407#issuecomment-1816363975
I have also checked that "Http Client" suffer of the same issue.
On the contrary, "REST Client" is working fine; however, the preemptive flag is ignored (always set to BASIC_PREEMPTIVE).
As a note, the "Http Post" and "Http Client" transformers use Apache Http 4.5 as the underlying HTTP client, while the 'REST Client' uses Glassfish Jersey.
---
In general, these 3 http-related transformers require some attention. What is the idea behind these transformers?
As far as I have understood:
- "Http Client" is a legacy for super simple scenarios
- "REST Client" follows the "Representational state transfer" style by making requests around resources. It supersedes 'Http Client' but does not allow multipart requests.
- "Http Post" covers, only for the POST method, the missing multipart feature of the previous transformer.
Therefore:
- "Http Client" usage should be discouraged in favor of the "REST Client" and maybe even be deprecated and removed later on
- 'Http Post' might be extended to support the PUT method and have underlying code/features aligned with "REST Client". As an example, the "Rest Client" accepts encrypted password as credentials, "Http Post" does not.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Bug]: HttpPost / Authorization does not work (hop)
Posted by "roy20021 (via GitHub)" <gi...@apache.org>.
roy20021 commented on issue #3407:
URL: https://github.com/apache/hop/issues/3407#issuecomment-1816458985
.take-issue
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org