From 5f7d30969299cd69b0b19928b364de2833ec3ab6 Mon Sep 17 00:00:00 2001 From: paulo-ocean Date: Wed, 26 Jun 2024 10:11:40 +0100 Subject: [PATCH] update computeStop, add some comments explaining the flow issues on the route --- src/services/Provider.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/services/Provider.ts b/src/services/Provider.ts index be0252bd..ca5f117a 100644 --- a/src/services/Provider.ts +++ b/src/services/Provider.ts @@ -644,7 +644,10 @@ export class Provider { let signatureMessage = consumerAddress signatureMessage += jobId || '' - // On provider service we just check signature owner + jobId + // On current provider impl (and nodes) we DO NOT check this signature + // On nodes we are signing again just the Nonce to send the request to Operator Service + // on current provider we sign: {owner}{job_id}{nonce}" OR {owner}{nonce} if no jobId + // On provider service STOP route, we just check signature owner + jobId OR just owner if no jobId // signatureMessage += (agreementId && `${this.noZeroX(agreementId)}`) || '' // signatureMessage += nonce const signature = await this.signProviderRequest(signer, signatureMessage)