<?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>14108</bug_id>
          
          <creation_ts>2020-10-26 04:03:34 +0000</creation_ts>
          <short_desc>Don&apos;t ignore certificates in wget fetcher</short_desc>
          <delta_ts>2021-08-12 14:59:50 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>BitBake</product>
          <component>bitbake</component>
          <version>3.3</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium+</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>3.4 M2</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ross Burton">ross.burton</reporter>
          <assigned_to name="Ross Burton">ross.burton</assigned_to>
          <cc>poky.bs.watcher</cc>
    
    <cc>poky.watcher</cc>
    
    <cc>randy.macleod</cc>
    
    <cc>tim.orling</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Yes (doc changes required)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>88485</commentid>
    <comment_count>0</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2020-10-26 04:03:34 +0000</bug_when>
    <thetext>Currently the http fetcher passes --no-check-certificate to wget, which means any SSL errors are ignored.

Whilst we have checksums for validation, this is not a good look and I believe we should remove this option.

I removed it locally and did a world fetch to a fresh DL_DIR of oe-core+meta-oe.  There were quite a few 404s from meta-oe, but nothing actually failed with SSL errors.

I&apos;d like to hope that we can simply remove the option, as any sites that have broken SSL can hopefully be switched to use http: instead.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88525</commentid>
    <comment_count>1</comment_count>
    <who name="Tim Orling">tim.orling</who>
    <bug_when>2020-10-30 05:57:52 +0000</bug_when>
    <thetext>But at least ignored certificates prevent man-in-the-middle attacks that straight http won&apos;t?

The real solution is all the upstreams need valid certificates ;)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88532</commentid>
    <comment_count>2</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2020-11-01 07:57:37 +0000</bug_when>
    <thetext>Good news: world fetch for oe-core+meta-oe resulted in no SSL failures.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88551</commentid>
    <comment_count>3</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2020-11-05 07:41:51 +0000</bug_when>
    <thetext>As per Joshua Watt: please allow certificate checking to be disabled globally because some companies use MITM edge devices that break wget.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90793</commentid>
    <comment_count>4</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2021-06-28 17:12:48 +0000</bug_when>
    <thetext>Patch on the list.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91169</commentid>
    <comment_count>5</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2021-08-12 14:59:50 +0000</bug_when>
    <thetext>Fixed in bitbake 4104850dd36096a9ff01836c5fca9ac0e452bcf8.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>