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

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>181009</bug_id>
          
          <creation_ts>2017-12-19 16:25:32 -0800</creation_ts>
          <short_desc>Cannot gather srflx or relay ICE candidates on IPv6 network (ICE agent hangs?)</short_desc>
          <delta_ts>2018-08-13 13:02:25 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebRTC</component>
          <version>Safari 11</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Mark Roberts">mroberts</reporter>
          <assigned_to name="youenn fablet">youennf</assigned_to>
          <cc>adam</cc>
    
    <cc>commit-queue</cc>
    
    <cc>eric.carlson</cc>
    
    <cc>svein</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>youennf</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1383403</commentid>
    <comment_count>0</comment_count>
    <who name="Mark Roberts">mroberts</who>
    <bug_when>2017-12-19 16:25:32 -0800</bug_when>
    <thetext>It appears that Safari (iOS and maybe macOS) may not be able to gather srflx and relay candidates from an IPv6 network.

Reproduction Steps:

1. Load https://jsfiddle.net/rn8b6y0k/1/ in Safari from an IPv6 network.*
2. Click &quot;Test without getUserMedia&quot;.

Expected Results:

  At least one srflx candidate is gathered.

Actual Results:

  No ICE candidates are gathered, &quot;null candidate&quot; is never received, and ICE connection state does not change.

* I tested using Safari on iOS with T-Mobile. I also tested Safari on macOS connected through my iPhone&apos;s T-Mobile hotspot. I got mixed results through the T-Mobile hotspot (I&apos;m not sure why). My colleague tried the same test on iOS using AT&amp;T, and it worked for him.

---

If you re-run this test with Chrome on macOS, it will succeed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1383411</commentid>
    <comment_count>1</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2017-12-19 16:35:24 -0800</bug_when>
    <thetext>I can reproduce the bug as well.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1383413</commentid>
    <comment_count>2</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2017-12-19 16:38:34 -0800</bug_when>
    <thetext>&lt;rdar://problem/36144555&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1383585</commentid>
    <comment_count>3</comment_count>
    <who name="sventy">svein</who>
    <bug_when>2017-12-20 03:33:55 -0800</bug_when>
    <thetext>If a device has both an IPv4 address and an IPv6 address, will this issue affect both?

In other words: Are no srflx and relay candidates gathered if the host is on a IPv6 network or does the issue affect only the IPv6 address?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1383690</commentid>
    <comment_count>4</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2017-12-20 09:32:32 -0800</bug_when>
    <thetext>(In reply to sventy from comment #3)
&gt; If a device has both an IPv4 address and an IPv6 address, will this issue
&gt; affect both?

Both host IPv4 and IPv6 should be gathered according my testing.

&gt; In other words: Are no srflx and relay candidates gathered if the host is on
&gt; a IPv6 network or does the issue affect only the IPv6 address?

I would think this only affects the IPv6 addresses.
My testing is limited at the moment though.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1406868</commentid>
    <comment_count>5</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2018-03-15 16:06:18 -0700</bug_when>
    <thetext>To be noted that on that page, stun.l.google.com is used and does not have any IPV6 address. It seems therefore strange that a srflx candidate would be gathered.

That said, we should not be timing out.
Maybe the DNS resolution is timing out and we do not properly handle this error case.
Or we are getting an IPv4 address and do not handle it properly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1419422</commentid>
    <comment_count>6</comment_count>
      <attachid>339224</attachid>
    <who name="youenn fablet">youennf</who>
    <bug_when>2018-05-01 14:22:12 -0700</bug_when>
    <thetext>Created attachment 339224
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1419684</commentid>
    <comment_count>7</comment_count>
      <attachid>339224</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2018-05-02 09:42:15 -0700</bug_when>
    <thetext>Comment on attachment 339224
Patch

Clearing flags on attachment: 339224

Committed r231235: &lt;https://trac.webkit.org/changeset/231235&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1419685</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2018-05-02 09:42:17 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1444749</commentid>
    <comment_count>9</comment_count>
    <who name="Mark Roberts">mroberts</who>
    <bug_when>2018-07-24 10:54:04 -0700</bug_when>
    <thetext>Thanks, Youenn. I haven&apos;t seen this mentioned in any recent Safari Tech Preview release notes. Is it out yet?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1449740</commentid>
    <comment_count>10</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2018-08-13 13:02:25 -0700</bug_when>
    <thetext>(In reply to Mark Roberts from comment #9)
&gt; Thanks, Youenn. I haven&apos;t seen this mentioned in any recent Safari Tech
&gt; Preview release notes. Is it out yet?

The patch should be out since quite some time on Safari Tech Preview and is probably in latest Mac/iOS betas.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>339224</attachid>
            <date>2018-05-01 14:22:12 -0700</date>
            <delta_ts>2018-05-02 09:42:15 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-181009-20180501142212.patch</filename>
            <type>text/plain</type>
            <size>2506</size>
            <attacher name="youenn fablet">youennf</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjMxMTk3CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IGI3NWJkY2E5NjdhNTA4NGVk
MjZmZWUwMWQ5ZTc5ZDU4NzA4Njk1MzEuLjA3YTBlZWU0MzQ5ZWExOGRjZGYzZDRmZWI5NGUwM2M2
ZGJmZWQwMTkgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjQgQEAKKzIwMTgtMDUtMDEgIFlvdWVubiBG
YWJsZXQgIDx5b3Vlbm5AYXBwbGUuY29tPgorCisgICAgICAgIENhbm5vdCBnYXRoZXIgc3JmbHgg
b3IgcmVsYXkgSUNFIGNhbmRpZGF0ZXMgb24gSVB2NiBuZXR3b3JrIChJQ0UgYWdlbnQgaGFuZ3M/
KQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTgxMDA5
CisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS8zNjE0NDU1NT4KKworICAgICAgICBSZXZpZXdlZCBi
eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBPbiBpT1MvSVB2NiBuZXR3b3JrcywgU1RVTiBz
ZXJ2ZXJzIG5hbWUgcmVzb2x1dGlvbiByZXR1cm5zIGEgemVybyBJUHY2IElQIGFkZHJlc3MuCisg
ICAgICAgIE5vIGVycm9yIGlzIHJhaXNlZCB3aGljaCBsZWFkcyB0byBzZW5kaW5nIFNUVU4gcmVx
dWVzdHMgd2l0aCB0aGF0IElQIGFkZHJlc3MuCisgICAgICAgIE9uY2UgdGhlIHJlcXVlc3QgdGlt
ZXMgb3V0LCB0aGUgSUNFIGNhbmRpZGF0ZSBnYXRoZXJpbmcgZmluaXNoZXMgd2l0aCBob3N0IGNh
bmRpZGF0ZXMgb25seS4KKworICAgICAgICBUaGlzIHBhdGNoIG1ha2VzIFdlYlJUQyBETlMgcmVz
b2x2ZXIgdG8gc2VuZCBvbmx5IElQdjQgcmVzb2x2ZWQgYWRkcmVzc2VzLgorICAgICAgICBTVFVO
IGlzIHVzZWQgZm9yIE5BVCB0cmF2ZXJzYWwgd2hpY2ggaXMgZm9yIElQdjQgYWRkcmVzc2VzLgor
ICAgICAgICBOb3Qgc2VuZGluZyBJUHY2IGFkZHJlc3NlcyBhbGxvd3MgdGVybWluYXRpbmcgSUNF
IGNhbmRpZGF0ZSBnYXRoZXJpbmcgc29vbmVyLgorCisgICAgICAgIE1hbnVhbGx5IHRlc3RlZCBv
biBpT1Mgd2l0aCBJUHY0L0lQdjYgYW5kIElQdjYgbmV0d29ya3MuCisKKyAgICAgICAgKiBOZXR3
b3JrUHJvY2Vzcy93ZWJydGMvTmV0d29ya1JUQ1Jlc29sdmVyQ29jb2EuY3BwOgorICAgICAgICAo
V2ViS2l0OjpyZXNvbHZlZE5hbWUpOgorCiAyMDE4LTA1LTAxICBPbGVrc2FuZHIgU2thY2hrb3Yg
IDxnc2thY2hrb3ZAZ21haWwuY29tPgogCiAgICAgICAgIFdlYkFzc2VtYmx5OiBhZGQgc3VwcG9y
dCBmb3Igc3RyZWFtIEFQSXMgLSBKYXZhU2NyaXB0IEFQSQpkaWZmIC0tZ2l0IGEvU291cmNlL1dl
YktpdC9OZXR3b3JrUHJvY2Vzcy93ZWJydGMvTmV0d29ya1JUQ1Jlc29sdmVyQ29jb2EuY3BwIGIv
U291cmNlL1dlYktpdC9OZXR3b3JrUHJvY2Vzcy93ZWJydGMvTmV0d29ya1JUQ1Jlc29sdmVyQ29j
b2EuY3BwCmluZGV4IDQ5NGRhZTQ0NmQ5YzUyYjE4Mjc1ODg4ZWE3Y2VjY2MyNDU2MjVlOWQuLjVm
OWI5ZGZiYTIzNDM2ZTQ0NDM2MmU4MzRkMGVkNjcyZmE5MDQ3OTYgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9XZWJLaXQvTmV0d29ya1Byb2Nlc3Mvd2VicnRjL05ldHdvcmtSVENSZXNvbHZlckNvY29hLmNw
cAorKysgYi9Tb3VyY2UvV2ViS2l0L05ldHdvcmtQcm9jZXNzL3dlYnJ0Yy9OZXR3b3JrUlRDUmVz
b2x2ZXJDb2NvYS5jcHAKQEAgLTU2LDcgKzU2LDEzIEBAIHN0YXRpYyB2b2lkIHJlc29sdmVkTmFt
ZShDRkhvc3RSZWYgaG9zdFJlZiwgQ0ZIb3N0SW5mb1R5cGUgdHlwZUluZm8sIGNvbnN0IENGU3Ry
CiAgICAgZm9yIChzaXplX3QgaW5kZXggPSAwOyBpbmRleCA8IGNvdW50OyArK2luZGV4KSB7CiAg
ICAgICAgIENGRGF0YVJlZiBkYXRhID0gKENGRGF0YVJlZilDRkFycmF5R2V0VmFsdWVBdEluZGV4
KHJlc29sdmVkQWRkcmVzc2VzLCBpbmRleCk7CiAgICAgICAgIGF1dG8qIGFkZHJlc3MgPSByZWlu
dGVycHJldF9jYXN0PGNvbnN0IHN0cnVjdCBzb2NrYWRkcl9pbio+KENGRGF0YUdldEJ5dGVQdHIo
ZGF0YSkpOwotICAgICAgICBhZGRyZXNzZXMudW5jaGVja2VkQXBwZW5kKFdlYkNvcmU6OklQQWRk
cmVzcygqYWRkcmVzcykpOworICAgICAgICBpZiAoYWRkcmVzcy0+c2luX2ZhbWlseSA9PSBBRl9J
TkVUKQorICAgICAgICAgICAgYWRkcmVzc2VzLnVuY2hlY2tlZEFwcGVuZChXZWJDb3JlOjpJUEFk
ZHJlc3MoKmFkZHJlc3MpKTsKKyAgICAgICAgLy8gRklYTUU6IFdlIHNob3VsZCBwcm9iYWJseSBy
ZXR1cm4gQUZfSU5FVDYgYWRkcmVzc2VzIGFzIHdlbGwuCisgICAgfQorICAgIGlmIChhZGRyZXNz
ZXMuaXNFbXB0eSgpKSB7CisgICAgICAgIHJlc29sdmVyLT5jb21wbGV0ZWQobWFrZVVuZXhwZWN0
ZWQoV2ViQ29yZTo6RE5TRXJyb3I6OkNhbm5vdFJlc29sdmUpKTsKKyAgICAgICAgcmV0dXJuOwog
ICAgIH0KICAgICByZXNvbHZlci0+Y29tcGxldGVkKFdURk1vdmUoYWRkcmVzc2VzKSk7CiB9Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>