Add authorization header in webhooks stored in secrets table

app/gen-server/lib/HomeDBManager.ts

@@ -1953,15 +1953,20 @@ export class HomeDBManager extends EventEmitter {
   // Update the webhook url in the webhook's corresponding secret (note: the webhook identifier is
   // its secret identifier).
   public async updateWebhookUrlAndAuth(
-    id: string, docId: string, url: string, auth: string | undefined, outerManager?: EntityManager) {
+    id: string, docId: string, url: string | undefined, auth: string | undefined, outerManager?: EntityManager) {
     return await this._runInTransaction(outerManager, async manager => {
       const value = await this.getSecret(id, docId, manager);
       if (!value) {
         throw new ApiError('Webhook with given id not found', 404);
       }
       const webhookSecret = JSON.parse(value);
-      webhookSecret.url = url;
-      webhookSecret.authorization = auth;
+      // update url and authorization only if not undefined to fit to patch calls
+      if (url !== undefined) {
+        webhookSecret.url = url;
+      }
+      if (auth !== undefined) {
+        webhookSecret.authorization = auth;
+      }
       await this.updateSecret(id, docId, JSON.stringify(webhookSecret), manager);
     });
   }

app/server/lib/DocApi.ts

@@ -936,7 +936,7 @@ export class DocWorkerApi {
 
         // update url and authorization header in homedb
         if (url || authorization) {
-          await this._dbManager.updateWebhookUrlAndAuth(webhookId, docId, url, authorization || undefined);
+          await this._dbManager.updateWebhookUrlAndAuth(webhookId, docId, url, authorization);
           activeDoc.triggers.webhookDeleted(webhookId); // clear cache
         }
 

test/server/lib/DocApi.ts

@@ -4961,6 +4961,8 @@ function testDocApi() {
 
           await check({isReadyColumn: null}, 200);
           await check({isReadyColumn: "bar"}, 404, `Column not found "bar"`);
+
+          await check({authorization: 'Bearer fake-token'}, 200);
         });
 
       });