Merge pull request #616 from nevermined-io/fix/credit_owner_download

Allowing credits asset publisher to download files

integration/external/Services_NFT1155.e2e.test.ts

@@ -206,7 +206,7 @@ describe('Gate-keeping of Web Services using NFT ERC-1155 End-to-End', () => {
       const isOperator = await subscriptionNFT.getContract.isOperator(transferNftCondition.address)
       assert.isTrue(isOperator)
 
-      subscriptionMetadata = getMetadata(undefined, 'Service Subscription NFT1155')
+      subscriptionMetadata = getMetadata(Math.random().toString(), 'Service Subscription NFT1155')
       subscriptionMetadata.main.type = 'subscription'
       const nftAttributes = NFTAttributes.getCreditsSubscriptionInstance({
         metadata: subscriptionMetadata,

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nevermined-io/sdk",
-  "version": "2.0.5",
+  "version": "2.0.6",
   "description": "Javascript SDK for connecting with Nevermined Data Platform ",
   "main": "./dist/node/sdk.js",
   "typings": "./dist/node/sdk.d.ts",

src/ddo/DDO.ts

@@ -24,6 +24,7 @@ import {
   DDOParamNotFoundError,
   DDOServiceAlreadyExists,
 } from '../errors/DDOError'
+import { jsonReplacer } from '../common'
 
 // DDO Services including a sales process
 export const SALES_SERVICES = ['access', 'compute', 'nft-sales']
@@ -41,7 +42,7 @@ export class DDO {
    * @returns DDO serialized.
    */
   public static serialize(ddo: DDO): string {
-    return JSON.stringify(ddo, null, 2)
+    return JSON.stringify(ddo, jsonReplacer, 2)
   }
 
   /**
@@ -244,7 +245,7 @@ export class DDO {
     const checksum = {}
     this.service.forEach((svc) => {
       checksum[svc.index] = this.checksum(
-        JSON.stringify(this.findServiceByType(svc.type).attributes.main),
+        JSON.stringify(this.findServiceByType(svc.type).attributes.main, jsonReplacer),
       )
     })
     return {

src/keeper/contracts/templates/NFTAccessTemplate.ts

@@ -163,6 +163,15 @@ export class NFTAccessTemplate extends BaseTemplate<NFTAccessTemplateParams, Ser
       return false
     }
 
+    const subscriptionOwner = await this.nevermined.assets.owner(ddo.id)
+    const consumerIsOwner =
+      params.consumer_address.toLowerCase() === subscriptionOwner.toLowerCase()
+
+    if (consumerIsOwner) {
+      // User calling is the asset owner so skipping track()
+      return false
+    }
+
     const nftAccessService = (
       params.service_index && params.service_index > 0
         ? ddo.findServiceByIndex(params.service_index)