<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>9439</bug_id>
          
          <creation_ts>2016-04-12 11:31:48 +0000</creation_ts>
          <short_desc>Verify that the REST API works with https</short_desc>
          <delta_ts>2016-07-15 19:45:38 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>6</classification_id>
          <classification>Yocto Project Subprojects</classification>
          <product>Layer Index</product>
          <component>Layer Index Metadata</component>
          <version>unspecified</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>OBSOLETE</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium+</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>2.2 M2</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Belen Barros Pena">belen.barros.pena</reporter>
          <assigned_to name="Elliot Smith">elliot.smith</assigned_to>
          <cc>bluelightning</cc>
    
    <cc>brian.avery</cc>
    
    <cc>mhalstead</cc>
          
          <qa_contact name="Libertad">libertad.gonzalez.de.la.cruz</qa_contact>
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>61079</commentid>
    <comment_count>0</comment_count>
    <who name="Belen Barros Pena">belen.barros.pena</who>
    <bug_when>2016-04-12 11:31:48 +0000</bug_when>
    <thetext>It was brought to our attention that the Layer Index does not have support for https. I believe Michael Halstead is pretty much ready to roll that out, but before he does we need to make sure Toaster can still retrieve information via the REST API when we switch to https by default.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61390</commentid>
    <comment_count>1</comment_count>
    <who name="Elliot Smith">elliot.smith</who>
    <bug_when>2016-04-19 11:24:05 +0000</bug_when>
    <thetext>Paul and Belen have tested this with Toaster.

I&apos;ve done a quick verification that urllib2 can access https sites (urllib2 is what we&apos;re using in LayerIndexLayerSource.update(), which fetches the layer index JSON).

However, we have some code in LayerIndexLayerSource which looks up the http_proxy env variable and makes use of this. If Toaster is using https to access the layer index in an env with proxies, we will need to get the https_proxy instead, so we may need some code changes here.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61391</commentid>
    <comment_count>2</comment_count>
    <who name="Elliot Smith">elliot.smith</who>
    <bug_when>2016-04-19 11:36:07 +0000</bug_when>
    <thetext>It looks like urllib2 automatically figures out the proxy settings; the code just looks them up for the purposes of printing debug output, so the code changes are minor.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61392</commentid>
    <comment_count>3</comment_count>
    <who name="Elliot Smith">elliot.smith</who>
    <bug_when>2016-04-19 12:22:11 +0000</bug_when>
    <thetext>Submitted a patch to show https_proxy in error output:
https://lists.yoctoproject.org/pipermail/toaster/2016-April/004469.html

The code needs no additional changes, as urllib2 automatically uses any http_proxy and https_proxy settings in the environment.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>63627</commentid>
    <comment_count>4</comment_count>
    <who name="Elliot Smith">elliot.smith</who>
    <bug_when>2016-07-04 15:38:25 +0000</bug_when>
    <thetext>Submitted v2 patch:
https://lists.yoctoproject.org/pipermail/toaster/2016-June/004904.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>63655</commentid>
    <comment_count>5</comment_count>
    <who name="Elliot Smith">elliot.smith</who>
    <bug_when>2016-07-05 15:46:27 +0000</bug_when>
    <thetext>Michael is making changing in this area of the code, including better capturing of errors, so marking this OBSOLETE.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>