<?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>193321</bug_id>
          
          <creation_ts>2019-01-10 07:41:04 -0800</creation_ts>
          <short_desc>[EME] Origin information in the instance session</short_desc>
          <delta_ts>2019-01-10 09:28:01 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebCore Misc.</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Xabier Rodríguez Calvar">calvaris</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>cturner</cc>
    
    <cc>eric.carlson</cc>
    
    <cc>jer.noble</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1493291</commentid>
    <comment_count>0</comment_count>
      <attachid>358791</attachid>
    <who name="Xabier Rodríguez Calvar">calvaris</who>
    <bug_when>2019-01-10 07:41:04 -0800</bug_when>
    <thetext>Created attachment 358791
propect patch

We&apos;re about to begin to implement license persistence and of course to comply with the spec (and common sense) we need to do things per-origin. I crafted this patch to provide origin information to the CDMInstanceSession so that it could communicate with CDMs should they require to perform persistence by themselves.

We are also wondering if we should provide persistence by ourselves and not rely on the CDMs for that so that we can allow users to clear information, without depending on CDMs. We don&apos;t even know if these two questions are unrelated and maybe we still need origin info in the CDMInstanceSession if we handle info by ourselves.

Input is appreciated.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1493319</commentid>
    <comment_count>1</comment_count>
    <who name="Jer Noble">jer.noble</who>
    <bug_when>2019-01-10 09:28:01 -0800</bug_when>
    <thetext>FYI, the way we handle this with FairPlay is to set the directory which the CDM is allowed to read from and write to on a per-origin basis, without exposing origin information directly to the CDM. I&apos;m not sure if this meets your per-origin goals, but that interface exists in CDMInstance and is implemented in CDMInstanceFairPlayStreamingAVFObjC::setStorageDirectory() if you want to TAL.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>358791</attachid>
            <date>2019-01-10 07:41:04 -0800</date>
            <delta_ts>2019-01-10 07:41:04 -0800</delta_ts>
            <desc>propect patch</desc>
            <filename>security-origin.patch</filename>
            <type>text/plain</type>
            <size>2222</size>
            <attacher name="Xabier Rodríguez Calvar">calvaris</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMvZW5jcnlwdGVkbWVkaWEvTWVkaWFL
ZXlTZXNzaW9uLmNwcCBiL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMvZW5jcnlwdGVkbWVkaWEvTWVk
aWFLZXlTZXNzaW9uLmNwcAppbmRleCA0YmVhOWEwMDg0Yi4uNGQ0ZjFiYWQzZDggMTAwNjQ0Ci0t
LSBhL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMvZW5jcnlwdGVkbWVkaWEvTWVkaWFLZXlTZXNzaW9u
LmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL2VuY3J5cHRlZG1lZGlhL01lZGlhS2V5
U2Vzc2lvbi5jcHAKQEAgLTY3MSw2ICs2NzEsMTMgQEAgdm9pZCBNZWRpYUtleVNlc3Npb246OnNl
bmRNZXNzYWdlKENETU1lc3NhZ2VUeXBlIG1lc3NhZ2VUeXBlLCBSZWY8U2hhcmVkQnVmZmVyPiYK
ICAgICBlbnF1ZXVlTWVzc2FnZShtZXNzYWdlVHlwZSwgbWVzc2FnZSk7CiB9CiAKK2NvbnN0IFNl
Y3VyaXR5T3JpZ2luJiBNZWRpYUtleVNlc3Npb246OnNlY3VyaXR5T3JpZ2luKCkKK3sKKyAgICBh
dXRvKiBkb2N1bWVudCA9IGRvd25jYXN0PERvY3VtZW50PihzY3JpcHRFeGVjdXRpb25Db250ZXh0
KCkpOworICAgIEFTU0VSVChkb2N1bWVudCk7CisgICAgcmV0dXJuIGRvY3VtZW50LT5zZWN1cml0
eU9yaWdpbigpOworfQorCiB2b2lkIE1lZGlhS2V5U2Vzc2lvbjo6dXBkYXRlRXhwaXJhdGlvbihk
b3VibGUpCiB7CiAgICAgbm90SW1wbGVtZW50ZWQoKTsKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJD
b3JlL01vZHVsZXMvZW5jcnlwdGVkbWVkaWEvTWVkaWFLZXlTZXNzaW9uLmggYi9Tb3VyY2UvV2Vi
Q29yZS9Nb2R1bGVzL2VuY3J5cHRlZG1lZGlhL01lZGlhS2V5U2Vzc2lvbi5oCmluZGV4IDM1ZDgx
NzQyNjhjLi5hNTI0NjA2OGRhOSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvTW9kdWxlcy9l
bmNyeXB0ZWRtZWRpYS9NZWRpYUtleVNlc3Npb24uaAorKysgYi9Tb3VyY2UvV2ViQ29yZS9Nb2R1
bGVzL2VuY3J5cHRlZG1lZGlhL01lZGlhS2V5U2Vzc2lvbi5oCkBAIC04Nyw2ICs4Nyw3IEBAIHBy
aXZhdGU6CiAgICAgLy8gQ0RNSW5zdGFuY2VTZXNzaW9uQ2xpZW50CiAgICAgdm9pZCB1cGRhdGVL
ZXlTdGF0dXNlcyhDRE1JbnN0YW5jZVNlc3Npb25DbGllbnQ6OktleVN0YXR1c1ZlY3RvciYmKSBv
dmVycmlkZTsKICAgICB2b2lkIHNlbmRNZXNzYWdlKENETU1lc3NhZ2VUeXBlLCBSZWY8U2hhcmVk
QnVmZmVyPiYmIG1lc3NhZ2UpIGZpbmFsOworICAgIGNvbnN0IFNlY3VyaXR5T3JpZ2luJiBzZWN1
cml0eU9yaWdpbigpIGZpbmFsOwogCiAgICAgLy8gRXZlbnRUYXJnZXQKICAgICBFdmVudFRhcmdl
dEludGVyZmFjZSBldmVudFRhcmdldEludGVyZmFjZSgpIGNvbnN0IG92ZXJyaWRlIHsgcmV0dXJu
IE1lZGlhS2V5U2Vzc2lvbkV2ZW50VGFyZ2V0SW50ZXJmYWNlVHlwZTsgfQpkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZW5jcnlwdGVkbWVkaWEvQ0RNSW5zdGFuY2VTZXNzaW9u
LmggYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9lbmNyeXB0ZWRtZWRpYS9DRE1JbnN0YW5jZVNl
c3Npb24uaAppbmRleCA3OTdhMmY5NWRhZC4uM2RlMGQyODI5ODAgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9XZWJDb3JlL3BsYXRmb3JtL2VuY3J5cHRlZG1lZGlhL0NETUluc3RhbmNlU2Vzc2lvbi5oCisr
KyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2VuY3J5cHRlZG1lZGlhL0NETUluc3RhbmNlU2Vz
c2lvbi5oCkBAIC0zOCw2ICszOCw3IEBACiBuYW1lc3BhY2UgV2ViQ29yZSB7CiAKIGNsYXNzIFNo
YXJlZEJ1ZmZlcjsKK2NsYXNzIFNlY3VyaXR5T3JpZ2luOwogCiBjbGFzcyBDRE1JbnN0YW5jZVNl
c3Npb25DbGllbnQgewogcHVibGljOgpAQCAtNDcsNiArNDgsNyBAQCBwdWJsaWM6CiAgICAgdXNp
bmcgS2V5U3RhdHVzVmVjdG9yID0gVmVjdG9yPHN0ZDo6cGFpcjxSZWY8U2hhcmVkQnVmZmVyPiwg
S2V5U3RhdHVzPj47CiAgICAgdmlydHVhbCB2b2lkIHVwZGF0ZUtleVN0YXR1c2VzKEtleVN0YXR1
c1ZlY3RvciYmKSA9IDA7CiAgICAgdmlydHVhbCB2b2lkIHNlbmRNZXNzYWdlKENETU1lc3NhZ2VU
eXBlLCBSZWY8U2hhcmVkQnVmZmVyPiYmIG1lc3NhZ2UpID0gMDsKKyAgICB2aXJ0dWFsIGNvbnN0
IFNlY3VyaXR5T3JpZ2luJiBzZWN1cml0eU9yaWdpbigpID0gMDsKIH07CiAKIGNsYXNzIENETUlu
c3RhbmNlU2Vzc2lvbiA6IHB1YmxpYyBSZWZDb3VudGVkPENETUluc3RhbmNlU2Vzc2lvbj4gewo=
</data>

          </attachment>
      

    </bug>

</bugzilla>