CLDSRV-224: return only once

Author: Seragonia

lib/api/bucketGetTagging.js

@@ -102,15 +102,15 @@ function bucketGetTagging(authInfo, request, log, callback) {
             });
             monitoring.promMetrics('GET', bucketName, err.code,
                 'getBucketTagging');
-            return callback(err, corsHeaders);
+        } else {
+            pushMetric('getBucketTagging', log, {
+                authInfo,
+                bucket: bucketName,
+            });
+            monitoring.promMetrics(
+                'GET', bucketName, '200', 'getBucketTagging');
         }
-        monitoring.promMetrics(
-            'GET', bucketName, '200', 'getBucketTagging');
-        pushMetric('getBucketTagging', log, {
-            authInfo,
-            bucket: bucketName,
-        });
-        return callback(null, xml, corsHeaders);
+        return callback(err, xml, corsHeaders);
     });
 }