feat: add ETag header to HEAD response and update tests

Author: larryrider

src/webdav/handlers/HEAD.handler.ts

@@ -19,6 +19,8 @@ export class HEADRequestHandler implements WebDavMethodHandler {
 
     webdavLogger.info(`[HEAD] [${driveItem.uuid}] Found Drive item`);
 
+    res.header('ETag', WebDavUtils.getItemETag(driveItem));
+
     if (driveItem.itemType === 'file') {
       const range = req.headers['range'];
       const rangeOptions = NetworkUtils.parseRangeHeader({

test/webdav/handlers/HEAD.handler.test.ts

@@ -40,6 +40,7 @@ describe('HEAD request handler', () => {
 
     await sut.handle(request, response);
     expect(response.status).toHaveBeenCalledWith(200);
+    expect(response.header).toHaveBeenCalledWith('ETag', WebDavUtils.getItemETag(mockFolder));
     expect(getRequestedResourceStub).toHaveBeenCalledOnce();
     expect(getFolderMetadataStub).toHaveBeenCalledOnce();
   });
@@ -68,6 +69,7 @@ describe('HEAD request handler', () => {
     expect(response.status).toHaveBeenCalledWith(200);
     expect(response.header).toHaveBeenCalledWith('Content-Type', 'application/octet-stream');
     expect(response.header).toHaveBeenCalledWith('Content-length', mockFile.size.toString());
+    expect(response.header).toHaveBeenCalledWith('ETag', WebDavUtils.getItemETag(mockFile));
     expect(getRequestedResourceStub).toHaveBeenCalledOnce();
     expect(getFileMetadataStub).toHaveBeenCalledOnce();
   });
@@ -100,6 +102,7 @@ describe('HEAD request handler', () => {
     expect(response.status).toHaveBeenCalledWith(200);
     expect(response.header).toHaveBeenCalledWith('Content-length', (mockSize - rangeStart).toString());
     expect(response.header).toHaveBeenCalledWith('Content-Type', 'application/octet-stream');
+    expect(response.header).toHaveBeenCalledWith('ETag', WebDavUtils.getItemETag(mockFile));
     expect(getRequestedResourceStub).toHaveBeenCalledOnce();
     expect(getFileMetadataStub).toHaveBeenCalledOnce();
   });