diff --git a/legacy/routes/header.js b/legacy/routes/header.js index 12f0bc29..71e8bc2a 100644 --- a/legacy/routes/header.js +++ b/legacy/routes/header.js @@ -8,18 +8,7 @@ var header = function (coverage, optionalCoverage) { optionalCoverage["HeaderParameterProtectedKey"] = 1; // NOT A THING optionalCoverage["CustomHeaderInRequest"] = 0; router.post("/param/:scenario", function (req, res, next) { - if (req.params.scenario === "existingkey") { - if (req.get("User-Agent") === "overwrite") { - coverage["HeaderParameterExistingKey"]++; - res.status(200).end(); - } else { - utils.send400( - res, - next, - 'Did not like scenario "' + req.params.scenario + '" with value ' + req.get("User-Agent"), - ); - } - } else if (req.params.scenario === "protectedkey") { + if (req.params.scenario === "protectedkey") { //We are making this change in the test server because for data plane we have to support scenarios where the user can override //the protected header like Content-Type by providing one. if (req.get("Content-Type") === "text/html") { @@ -37,15 +26,6 @@ var header = function (coverage, optionalCoverage) { } }); - router.post("/custom/x-ms-client-request-id/9C4D50EE-2D56-4CD3-8152-34347DC9F2B0", function (req, res, next) { - if (req.get("x-ms-client-request-id").toLowerCase() === "9C4D50EE-2D56-4CD3-8152-34347DC9F2B0".toLowerCase()) { - optionalCoverage["CustomHeaderInRequest"]++; - res.status(200).end(); - } else { - utils.send400(res, next, 'Did not like client request id "' + req.get("x-ms-client-request-id")); - } - }); - router.post("/response/:scenario", function (req, res, next) { if (req.params.scenario === "existingkey") { coverage["HeaderResponseExistingKey"]++; diff --git a/package.json b/package.json index 417e9eab..b2aca885 100644 --- a/package.json +++ b/package.json @@ -32,7 +32,7 @@ "format": "npm run -s prettier -- --write", "check-format": "npm run -s prettier -- --check", "prettier": "prettier --config ./.prettierrc.yml **/*.{ts,js,cjs,mjs,json,yml,yaml,md}", - "validate-spec-coverage": "node ./dist/cli/cli.js validate-spec-coverage --maxErrorCount=77" + "validate-spec-coverage": "node ./dist/cli/cli.js validate-spec-coverage --maxErrorCount=76" }, "engines": { "node": ">=10" @@ -92,4 +92,4 @@ "xml2js": "^0.4.23", "yargs": "^17.3.1" } -} +} \ No newline at end of file diff --git a/swagger/examples/header_paramExistingKey.json b/swagger/examples/header_paramExistingKey.json deleted file mode 100644 index 896013db..00000000 --- a/swagger/examples/header_paramExistingKey.json +++ /dev/null @@ -1,8 +0,0 @@ -{ - "parameters": { - "User-Agent": "overwrite" - }, - "responses": { - "200": {} - } -} diff --git a/swagger/header.json b/swagger/header.json index 909cc953..8a36dff6 100644 --- a/swagger/header.json +++ b/swagger/header.json @@ -10,38 +10,6 @@ "produces": ["application/json"], "consumes": ["application/json"], "paths": { - "/header/param/existingkey": { - "post": { - "operationId": "header_paramExistingKey", - "description": "Send a post request with header value \"User-Agent\": \"overwrite\"", - "x-ms-examples": { - "header_paramExistingKey": { - "$ref": "./examples/header_paramExistingKey.json" - } - }, - "tags": ["Header Operations"], - "parameters": [ - { - "description": "Send a post request with header value \"User-Agent\": \"overwrite\"", - "name": "User-Agent", - "in": "header", - "type": "string", - "required": true - } - ], - "responses": { - "200": { - "description": "Empty Response" - }, - "default": { - "description": "Unexpected error", - "schema": { - "$ref": "#/definitions/Error" - } - } - } - } - }, "/header/response/existingkey": { "post": { "operationId": "header_responseExistingKey",