Bug 607799
| Summary: | The "JBoss CSP Patch Feed" is not authenticating to the default Feed URL | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Other] RHQ Project | Reporter: | John Sefler <jsefler> | ||||
| Component: | Content | Assignee: | Simeon Pinder <spinder> | ||||
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Corey Welton <cwelton> | ||||
| Severity: | urgent | Docs Contact: | |||||
| Priority: | urgent | ||||||
| Version: | 3.0.0 | CC: | rtimaniy | ||||
| Target Milestone: | --- | ||||||
| Target Release: | --- | ||||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | 2.4 | Doc Type: | Bug Fix | ||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2010-08-12 16:49:55 UTC | Type: | --- | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Bug Depends On: | |||||||
| Bug Blocks: | 562434, 593121, 603787 | ||||||
| Attachments: |
|
||||||
|
Description
John Sefler
2010-06-24 21:12:45 UTC
Simeon Can you confirm this problem and investigate what it would take to support the 301 redirect request we get back from the CSP Thanks Yes. This is a problem and I've confirmed the described behavior. I'm looking into what the fix should be. Browsers handle the redirect seamlessly, we need to do the same in the code. This is some odd behavior with the HttpClient. The HttpClient knows how to redirect properly and loads the redirect url as one would expect but does not update the Method instance properly and throws Http 401 unauthorized responses(even with correct credentials applied) if we proceed with the redirected instance. The fix is to store the original and actually loaded urls and reinitialize the instance when they differ. I log a warning to server for the admin to update the original url if the redirection is permanent. I looked into sending a message to the UI, but this would require additional changes that might introduce unnecessary instability. Available in successful builds of release branch release-3.0.0 >= 108. commit hash: 5055dca9b0d0bfde336be48704d50628663bd804 ## It's likely that this needs to be made available for patches to 2.3 and 2.3.1(not sure if further back, this class starts after 2.3) or at least a notification should be made that customers need to change the content source url, should be made. ## Verified on JON 2.4 GA_QA (tag-jon-release build# 49) Revision: 10771 Steps: 1) Administration > Content > Content Sources > JBoss CSP Patch Feed 2) edit the Username and Password to a valid account 3) edit Active to Yes 4) click TEST CONNECTION 5) click SYNCHRONIZE Observation: 'TEST CONNECTION' successful with the message: 'Test passed - the remote repository for [JBoss CSP Patch Feed] is available. ' 'SYNCHRONIZE' gives message: 'Synchronizing content source [JBoss CSP Patch Feed] now. ' and list of available packages. Created attachment 429190 [details]
HTTP client log.
Mass-closure of verified bugs against JON. |