DID method creates new Ethr-DID without passing singer or private key

Description In the documentation, it is mentioned that ‘either signer or privateKey’ is required to create a new Ethr-DID. However, a new Ethr-Did can be created just passing the Ethereum address.

To Reproduce

1. init npm project
2. paste the following code into inde.js
3. configure the provider
4. install the dependencies ethjs-provider-http@0.1.6 and ethr-did@1.1.0
5. run node index.js

Sample Code

const HttpProvider = require('ethjs-provider-http')
const provider = new HttpProvider('http://127.0.0.1:8545')

const EthrDID = require('ethr-did');

let createDid = async () => {

    const address = '0xCF402F0891f9551eA0e2cEE7A7e491C4e83Fc079';
    const ethrDid = new EthrDID({ address });
    console.log(ethrDid);

}

createDid()

Expected behavior It should throw an error stating ‘either signer or private key is required’.

Actual behavior It successfully creates a new did.

Screenshots

Dependencies:

• Node.js: 10.16.3
• ethjs-provider-http: ^0.1.6
• ethr-did: ^1.1.0

[m-yahya]

Thank you very much for your explanation.

[mirceanis]

It was at the time of writing the resolver. However, the W3C spec regarding the public keys has changed. The owner is no longer a recognized property of a public key. The spec describes a controller property but that may have slightly different semantics.

I suggest not using the owner property of the resolved document at this time/version. If your use case requires that you find the owner as you expect, it is probably safer to call the registry contract directly instead of relying on the property that may disappear in a future version.

Thanks for spotting this