Bug 3240 - [Hob] Hob does not properly handles setting proxies
Summary: [Hob] Hob does not properly handles setting proxies
Status: VERIFIED FIXED
Alias: None
Product: Hob
Classification: Build System, Metadata & Runtime
Component: hob (show other bugs)
Version: 1.3
Hardware: x86 Multiple
: Medium normal
Target Milestone: 1.4
Assignee: Cristian Iorga
QA Contact:
URL:
Whiteboard: 30d1943a79b8012609cab9f71cc3a36a6ce4543e
Depends on: 3026
Blocks:
  Show dependency tree
 
Reported: 2012-10-05 10:44 UTC by Cristian Iorga
Modified: 2013-07-15 09:46 UTC (History)
6 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Cristian Iorga 2012-10-05 10:44:24 UTC 
Hob does not properly handles setting proxies

Steps to reproduce:

1. Remove or rename .hob directory in the build directory.
2. Unset http_proxy and HTTP_PROXY in a shell script.
3. Start hob from the same shell session
4. Go to Settings -> Proxies
5. Notice that HTTP proxy settings should be empty
6. Edit HTTP proxy by adding a proxy
7. Push the related Details button and close it
8. Previously set HTTP proxy vanished.

Notice that this situation does not normally happen in a properly set environment regarding proxies, but it can be easily reproduced in the Build Appliance (where no proxies are set when Hob starts).
Also, it is possible to be reproducible for any proxy, not only for the HTTP one.
Comment 1 Cristian Iorga 2012-10-15 11:04:23 UTC 
Part of HOB improvement sprint for 1.4
Comment 2 Cristian Iorga 2012-10-16 09:22:04 UTC 
In development
Comment 3 Cristian Iorga 2012-10-16 11:26:48 UTC 
The fix is dependent on the fix for bug 3026.
The fix for YB #3026 have been submitted, but not yet merged.
Comment 4 Cristian Iorga 2012-10-26 06:17:28 UTC 
Fix for YB #3026 is now merged in YP master branch.
Comment 5 Cristian Iorga 2012-10-26 09:26:17 UTC 
Patch submitted for review.
Comment 6 Cristian Iorga 2012-10-29 08:46:07 UTC 
Fixed with commit 30d1943a79b8012609cab9f71cc3a36a6ce4543e.
Comment 7 Andreea Brandusa Proca 2013-07-15 09:46:23 UTC 
Verified in master:a63229917a5708de2d161aba0d67168ce0da6365.