{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":19537979,"defaultBranch":"master","name":"tyk","ownerLogin":"TykTechnologies","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2014-05-07T14:38:24.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/16121529?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1716480248.0","currentOid":""},"activityList":{"items":[{"before":"d27a42ffcd3b78aa859c9ae29ad0e9529a61c6ce","after":"9154430b8ac2659e50d68ef029cd65921ea57ec5","ref":"refs/heads/feature/streaming","pushedAt":"2024-05-23T17:07:36.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"buger","name":"Leonid Bugaev","path":"/buger","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/14009?s=80&v=4"},"commit":{"message":"Add portal support","shortMessageHtmlLink":"Add portal support"}},{"before":"149349fb1dd1a47df5ffa2da4aae3cb6c066fb15","after":"74c8492ba4fa78d4af8869b402b25009f5c34b5f","ref":"refs/heads/repro/tt-11953/reproduce-build-issue","pushedAt":"2024-05-23T16:49:30.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"},"commit":{"message":"Use golang docker image for main dockerfile, still not failing","shortMessageHtmlLink":"Use golang docker image for main dockerfile, still not failing"}},{"before":null,"after":"dc1f95195102cd226999ad2909258e70a971a732","ref":"refs/heads/feat/TT-9462/tag-cached-response","pushedAt":"2024-05-23T16:04:08.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"joshblakeley","name":"Josh Blakeley","path":"/joshblakeley","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/31618778?s=80&v=4"},"commit":{"message":"add tag for cached-response when response is cached","shortMessageHtmlLink":"add tag for cached-response when response is cached"}},{"before":"bfb740afc51df0a76cb535cbfa8a5faaa28bc1ba","after":"149349fb1dd1a47df5ffa2da4aae3cb6c066fb15","ref":"refs/heads/repro/tt-11953/reproduce-build-issue","pushedAt":"2024-05-23T14:00:39.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"},"commit":{"message":"Add debugging commands into taskfiles, update makefile, dockerfile, goreleaser","shortMessageHtmlLink":"Add debugging commands into taskfiles, update makefile, dockerfile, g…"}},{"before":null,"after":"bfb740afc51df0a76cb535cbfa8a5faaa28bc1ba","ref":"refs/heads/repro/tt-11953/reproduce-build-issue","pushedAt":"2024-05-23T13:59:00.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"},"commit":{"message":"Add debugging commands into taskfiles, update makefile, dockerfile, goreleaser","shortMessageHtmlLink":"Add debugging commands into taskfiles, update makefile, dockerfile, g…"}},{"before":null,"after":"46b54fa9f39391bfec7227d2cebc65c38e54192c","ref":"refs/heads/feat/TT-11997/Backend-logic-for-request_headers_rewrite","pushedAt":"2024-05-23T13:31:00.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"buraksezer","name":"Burak Sezer","path":"/buraksezer","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/33702537?s=80&v=4"},"commit":{"message":"[TT-11997] Backend logic for request_headers_rewrite","shortMessageHtmlLink":"[TT-11997] Backend logic for request_headers_rewrite"}},{"before":"1d4aa75eecab9cf9c7e831d8ff3947645890a54d","after":"cbcc2ad63146791352171d6bf0cac535c60bec9f","ref":"refs/heads/TT-11470-human-readable-info","pushedAt":"2024-05-23T12:48:09.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"padiazg","name":"Patricio Díaz ","path":"/padiazg","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/15754252?s=80&v=4"},"commit":{"message":"fix missing references","shortMessageHtmlLink":"fix missing references"}},{"before":"0f0290a89ff09081c99b9314b68b2d1c8de2735b","after":"c09286fd84be114d1745f8eb84d841e4733690eb","ref":"refs/heads/TT-12095","pushedAt":"2024-05-23T12:41:30.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"mativm02","name":"Matias","path":"/mativm02","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/83959431?s=80&v=4"},"commit":{"message":"checking if hashing is enabled before creating span","shortMessageHtmlLink":"checking if hashing is enabled before creating span"}},{"before":"babdf44cd98b4109c6a50a9f29eb58bd285fb4f5","after":"1d4aa75eecab9cf9c7e831d8ff3947645890a54d","ref":"refs/heads/TT-11470-human-readable-info","pushedAt":"2024-05-23T12:27:40.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"padiazg","name":"Patricio Díaz ","path":"/padiazg","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/15754252?s=80&v=4"},"commit":{"message":"renaned internal/models to internal/model","shortMessageHtmlLink":"renaned internal/models to internal/model"}},{"before":"5dda8b15558bb831a1eb9488e9f57c5e4092bf27","after":"d27a42ffcd3b78aa859c9ae29ad0e9529a61c6ce","ref":"refs/heads/feature/streaming","pushedAt":"2024-05-23T12:11:55.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"buger","name":"Leonid Bugaev","path":"/buger","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/14009?s=80&v=4"},"commit":{"message":"Fix bugs","shortMessageHtmlLink":"Fix bugs"}},{"before":"a5bd292ace6d85f06c99d9283af1f8587c4673f6","after":null,"ref":"refs/heads/feat/TT-11998/add-graphql-proxy-features-section","pushedAt":"2024-05-23T12:01:31.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"pvormste","name":"Patric Vormstein","path":"/pvormste","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4503955?s=80&v=4"}},{"before":"11d5798b31e49f8d28a2262925b9440d9e86c58a","after":"e1ea94a5f4e5ea8029b2e837171f0bddbe110b6b","ref":"refs/heads/master","pushedAt":"2024-05-23T12:01:30.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"pvormste","name":"Patric Vormstein","path":"/pvormste","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4503955?s=80&v=4"},"commit":{"message":"add features section to graphql proxy config (#6298)\n\n### **User description**\r\nThis PR adds a new `features` section to the GraphQL proxy\r\nconfiguration. It will allow the enable or disable specific features for\r\nproxy-only mode.\r\n\r\n## Types of changes\r\n\r\n<!-- What types of changes does your code introduce? Put an `x` in all\r\nthe boxes that apply: -->\r\n\r\n- [ ] Bug fix (non-breaking change which fixes an issue)\r\n- [x] New feature (non-breaking change which adds functionality)\r\n- [ ] Breaking change (fix or feature that would cause existing\r\nfunctionality to change)\r\n- [ ] Refactoring or add test (improvements in base code or adds test\r\ncoverage to functionality)\r\n\r\n\r\n___\r\n\r\n### **PR Type**\r\nEnhancement\r\n\r\n\r\n___\r\n\r\n### **Description**\r\nThis PR introduces a new `features` section to the GraphQL proxy\r\nconfiguration:\r\n- Added `features` section to `GraphQLProxyConfig` in\r\n`apidef/api_definitions.go`.\r\n- Introduced `GraphQLProxyFeaturesConfig` struct with\r\n`UseImmutableHeaders` field.\r\n- Updated `DummyAPI` function to include default `features`\r\nconfiguration.\r\n- Updated GraphQL proxy schema in `apidef/schema.go` to include\r\n`features` section with `use_immutable_headers` property.\r\n\r\n\r\n___\r\n\r\n\r\n\r\n### **Changes walkthrough** 📝\r\n<table><thead><tr><th></th><th align=\"left\">Relevant\r\nfiles</th></tr></thead><tbody><tr><td><strong>Enhancement\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>api_definitions.go</strong><dd><code>Add features\r\nsection to GraphQL proxy configuration.</code>&nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/api_definitions.go\r\n<li>Added <code>features</code> section to\r\n<code>GraphQLProxyConfig</code>.<br> <li> Introduced\r\n<code>GraphQLProxyFeaturesConfig</code> struct with\r\n<code>UseImmutableHeaders</code> <br>field.<br> <li> Updated\r\n<code>DummyAPI</code> function to include default <code>features</code>\r\nconfiguration.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6298/files#diff-9961ccc89a48d32db5b47ba3006315ef52f6e5007fb4b09f8c5d6d299c669d67\">+10/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>schema.go</strong><dd><code>Update GraphQL proxy schema\r\nwith features section.</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/schema.go\r\n<li>Added <code>features</code> section to GraphQL proxy schema.<br>\r\n<li> Included <code>use_immutable_headers</code> property in the\r\nschema.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6298/files#diff-f8a37bb370eb6fe20063786a5e6ea3d85a5c91d8e289f0b3e045830c4d322095\">+8/-0</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n</table></td></tr></tr></tbody></table>\r\n\r\n___\r\n\r\n> 💡 **PR-Agent usage**:\r\n>Comment `/help` on the PR to get a list of all available PR-Agent tools\r\nand their descriptions","shortMessageHtmlLink":"add features section to graphql proxy config (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"2310494524\" data-permission-text=\"Title is private\" data-url=\"https://github.com/TykTechnologies/tyk/issues/6298\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/TykTechnologies/tyk/pull/6298/hovercard\" href=\"https://github.com/TykTechnologies/tyk/pull/6298\">#6298</a>)"}},{"before":"9e3f744c538cac4a78a3d11f9867387d42b1c968","after":"a5bd292ace6d85f06c99d9283af1f8587c4673f6","ref":"refs/heads/feat/TT-11998/add-graphql-proxy-features-section","pushedAt":"2024-05-23T11:41:30.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"pvormste","name":"Patric Vormstein","path":"/pvormste","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4503955?s=80&v=4"},"commit":{"message":"fix another broken test","shortMessageHtmlLink":"fix another broken test"}},{"before":"182a0dde6277bd377352dcb7cde7c6796db868da","after":"9e3f744c538cac4a78a3d11f9867387d42b1c968","ref":"refs/heads/feat/TT-11998/add-graphql-proxy-features-section","pushedAt":"2024-05-23T11:17:47.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"pvormste","name":"Patric Vormstein","path":"/pvormste","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4503955?s=80&v=4"},"commit":{"message":"fix another broken test","shortMessageHtmlLink":"fix another broken test"}},{"before":"09e0d8d8fccb88352c5dd106bda146fed5978d8f","after":"182a0dde6277bd377352dcb7cde7c6796db868da","ref":"refs/heads/feat/TT-11998/add-graphql-proxy-features-section","pushedAt":"2024-05-23T09:23:44.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"pvormste","name":"Patric Vormstein","path":"/pvormste","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4503955?s=80&v=4"},"commit":{"message":"fix broken test","shortMessageHtmlLink":"fix broken test"}},{"before":"03c1a85147bde7b07cbe134f5b718d43efa3df49","after":null,"ref":"refs/heads/merge/release-5.3/11d5798b31e49f8d28a2262925b9440d9e86c58a","pushedAt":"2024-05-23T09:16:14.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"buger","name":"Leonid Bugaev","path":"/buger","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/14009?s=80&v=4"}},{"before":"a84de17d3af1e8cdd4ead00d429e7a467f7606e0","after":"c9ff51d83a47e8b1f3b1431ebc9f8802e547422e","ref":"refs/heads/release-5.3","pushedAt":"2024-05-23T09:16:13.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"buger","name":"Leonid Bugaev","path":"/buger","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/14009?s=80&v=4"},"commit":{"message":"Merging to release-5.3: [TT-12193] Fix poor error handling in webhook event templates (#6303)\n\n[TT-12193] Fix poor error handling in webhook event templates (#6303)\n\n### **User description**\r\nFixes error handling in webhook handler templates\r\n\r\nhttps://tyktech.atlassian.net/browse/TT-12193\r\n\r\n___\r\n\r\n### **PR Type**\r\nBug fix, Tests\r\n\r\n\r\n___\r\n\r\n### **Description**\r\n- Added error handling for template execution in the `CreateBody` method\r\nof `WebHookHandler`.\r\n- Refactored test helper function to include error handling.\r\n- Replaced panic and error checks with `assert.NoError` in tests.\r\n- Improved error messages in tests for better clarity.\r\n\r\n\r\n___\r\n\r\n\r\n\r\n### **Changes walkthrough** 📝\r\n<table><thead><tr><th></th><th align=\"left\">Relevant\r\nfiles</th></tr></thead><tbody><tr><td><strong>Bug fix\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>event_handler_webhooks.go</strong><dd><code>Add error\r\nhandling in `CreateBody` method of WebHookHandler</code></dd></summary>\r\n<hr>\r\n\r\ngateway/event_handler_webhooks.go\r\n<li>Added error handling for template execution in\r\n<code>CreateBody</code> method.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6303/files#diff-6587ad3f2629cfa6c84a71144127acd6cc7824e5141f0b4961848945a87e0198\">+4/-1</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n</table></td></tr><tr><td><strong>Tests\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n\r\n<summary><strong>event_handler_webhooks_test.go</strong><dd><code>Improve\r\nerror handling and assertions in webhook handler\r\ntests</code></dd></summary>\r\n<hr>\r\n\r\ngateway/event_handler_webhooks_test.go\r\n<li>Refactored test helper function to include error handling.<br> <li>\r\nReplaced panic and error checks with <code>assert.NoError</code>.<br>\r\n<li> Improved error messages in tests.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6303/files#diff-1a2f8d6ab4443031b0f8486809453f6389291a6f625745fe8c65ac5cdb0e9621\">+22/-24</a>&nbsp;\r\n</td>\r\n</tr>                    \r\n</table></td></tr></tr></tbody></table>\r\n\r\n___\r\n\r\n> 💡 **PR-Agent usage**:\r\n>Comment `/help` on the PR to get a list of all available PR-Agent tools\r\nand their descriptions\r\n\r\nCo-authored-by: Tit Petric <tit@tyk.io>","shortMessageHtmlLink":"Merging to release-5.3: [TT-12193] Fix poor error handling in webhook…"}},{"before":"f61fcf357ed62301cd0042b276c57cdfb1c9d65c","after":"03c1a85147bde7b07cbe134f5b718d43efa3df49","ref":"refs/heads/merge/release-5.3/11d5798b31e49f8d28a2262925b9440d9e86c58a","pushedAt":"2024-05-23T08:55:44.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"},"commit":{"message":"Assert was missing from event handler webhooks test","shortMessageHtmlLink":"Assert was missing from event handler webhooks test"}},{"before":"2e1d24f06df4d7b966bbed9200932eb9720f70a5","after":"09e0d8d8fccb88352c5dd106bda146fed5978d8f","ref":"refs/heads/feat/TT-11998/add-graphql-proxy-features-section","pushedAt":"2024-05-23T08:32:48.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"pvormste","name":"Patric Vormstein","path":"/pvormste","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4503955?s=80&v=4"},"commit":{"message":"add features section to graphql proxy config","shortMessageHtmlLink":"add features section to graphql proxy config"}},{"before":null,"after":"f61fcf357ed62301cd0042b276c57cdfb1c9d65c","ref":"refs/heads/merge/release-5.3/11d5798b31e49f8d28a2262925b9440d9e86c58a","pushedAt":"2024-05-23T08:27:12.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"tykbot[bot]","name":null,"path":"/apps/tykbot","primaryAvatarUrl":"https://avatars.githubusercontent.com/in/35290?s=80&v=4"},"commit":{"message":"[TT-12193] Fix poor error handling in webhook event templates (#6303)\n\n### **User description**\r\nFixes error handling in webhook handler templates\r\n\r\nhttps://tyktech.atlassian.net/browse/TT-12193\r\n\r\n___\r\n\r\n### **PR Type**\r\nBug fix, Tests\r\n\r\n\r\n___\r\n\r\n### **Description**\r\n- Added error handling for template execution in the `CreateBody` method\r\nof `WebHookHandler`.\r\n- Refactored test helper function to include error handling.\r\n- Replaced panic and error checks with `assert.NoError` in tests.\r\n- Improved error messages in tests for better clarity.\r\n\r\n\r\n___\r\n\r\n\r\n\r\n### **Changes walkthrough** 📝\r\n<table><thead><tr><th></th><th align=\"left\">Relevant\r\nfiles</th></tr></thead><tbody><tr><td><strong>Bug fix\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>event_handler_webhooks.go</strong><dd><code>Add error\r\nhandling in `CreateBody` method of WebHookHandler</code></dd></summary>\r\n<hr>\r\n\r\ngateway/event_handler_webhooks.go\r\n<li>Added error handling for template execution in\r\n<code>CreateBody</code> method.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6303/files#diff-6587ad3f2629cfa6c84a71144127acd6cc7824e5141f0b4961848945a87e0198\">+4/-1</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n</table></td></tr><tr><td><strong>Tests\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n\r\n<summary><strong>event_handler_webhooks_test.go</strong><dd><code>Improve\r\nerror handling and assertions in webhook handler\r\ntests</code></dd></summary>\r\n<hr>\r\n\r\ngateway/event_handler_webhooks_test.go\r\n<li>Refactored test helper function to include error handling.<br> <li>\r\nReplaced panic and error checks with <code>assert.NoError</code>.<br>\r\n<li> Improved error messages in tests.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6303/files#diff-1a2f8d6ab4443031b0f8486809453f6389291a6f625745fe8c65ac5cdb0e9621\">+22/-24</a>&nbsp;\r\n</td>\r\n</tr>                    \r\n</table></td></tr></tr></tbody></table>\r\n\r\n___\r\n\r\n> 💡 **PR-Agent usage**:\r\n>Comment `/help` on the PR to get a list of all available PR-Agent tools\r\nand their descriptions\r\n\r\nCo-authored-by: Tit Petric <tit@tyk.io>\n(cherry picked from commit 11d5798b31e49f8d28a2262925b9440d9e86c58a)","shortMessageHtmlLink":"[TT-12193] Fix poor error handling in webhook event templates (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"2312148465\" data-permission-text=\"Title is private\" data-url=\"https://github.com/TykTechnologies/tyk/issues/6303\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/TykTechnologies/tyk/pull/6303/hovercard\" href=\"https://github.com/TykTechnologies/tyk/pull/6303\">#6303</a>)"}},{"before":"fe24bf0824f17f74906d14c707e96fcf9e51f952","after":null,"ref":"refs/heads/fix/event-handler-webhooks-error-handling","pushedAt":"2024-05-23T08:24:48.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"}},{"before":"7cca61a0d65fd113a2abf2ed9c66bd18830f797d","after":"11d5798b31e49f8d28a2262925b9440d9e86c58a","ref":"refs/heads/master","pushedAt":"2024-05-23T08:24:47.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"},"commit":{"message":"[TT-12193] Fix poor error handling in webhook event templates (#6303)\n\n### **User description**\r\nFixes error handling in webhook handler templates\r\n\r\nhttps://tyktech.atlassian.net/browse/TT-12193\r\n\r\n___\r\n\r\n### **PR Type**\r\nBug fix, Tests\r\n\r\n\r\n___\r\n\r\n### **Description**\r\n- Added error handling for template execution in the `CreateBody` method\r\nof `WebHookHandler`.\r\n- Refactored test helper function to include error handling.\r\n- Replaced panic and error checks with `assert.NoError` in tests.\r\n- Improved error messages in tests for better clarity.\r\n\r\n\r\n___\r\n\r\n\r\n\r\n### **Changes walkthrough** 📝\r\n<table><thead><tr><th></th><th align=\"left\">Relevant\r\nfiles</th></tr></thead><tbody><tr><td><strong>Bug fix\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>event_handler_webhooks.go</strong><dd><code>Add error\r\nhandling in `CreateBody` method of WebHookHandler</code></dd></summary>\r\n<hr>\r\n\r\ngateway/event_handler_webhooks.go\r\n<li>Added error handling for template execution in\r\n<code>CreateBody</code> method.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6303/files#diff-6587ad3f2629cfa6c84a71144127acd6cc7824e5141f0b4961848945a87e0198\">+4/-1</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n</table></td></tr><tr><td><strong>Tests\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n\r\n<summary><strong>event_handler_webhooks_test.go</strong><dd><code>Improve\r\nerror handling and assertions in webhook handler\r\ntests</code></dd></summary>\r\n<hr>\r\n\r\ngateway/event_handler_webhooks_test.go\r\n<li>Refactored test helper function to include error handling.<br> <li>\r\nReplaced panic and error checks with <code>assert.NoError</code>.<br>\r\n<li> Improved error messages in tests.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6303/files#diff-1a2f8d6ab4443031b0f8486809453f6389291a6f625745fe8c65ac5cdb0e9621\">+22/-24</a>&nbsp;\r\n</td>\r\n</tr>                    \r\n</table></td></tr></tr></tbody></table>\r\n\r\n___\r\n\r\n> 💡 **PR-Agent usage**:\r\n>Comment `/help` on the PR to get a list of all available PR-Agent tools\r\nand their descriptions\r\n\r\nCo-authored-by: Tit Petric <tit@tyk.io>","shortMessageHtmlLink":"[TT-12193] Fix poor error handling in webhook event templates (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"2312148465\" data-permission-text=\"Title is private\" data-url=\"https://github.com/TykTechnologies/tyk/issues/6303\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/TykTechnologies/tyk/pull/6303/hovercard\" href=\"https://github.com/TykTechnologies/tyk/pull/6303\">#6303</a>)"}},{"before":"90653b2b40368b836dd1ef1650c10a58a838b0a4","after":null,"ref":"refs/heads/merge/release-5.3/63963725d1b802d03841651f290d1cbd11f398e9","pushedAt":"2024-05-23T08:17:09.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"}},{"before":"0038992886cd297e7aabeb473aa25d6e180557bb","after":"a84de17d3af1e8cdd4ead00d429e7a467f7606e0","ref":"refs/heads/release-5.3","pushedAt":"2024-05-23T08:17:08.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"},"commit":{"message":"Merging to release-5.3: [TT-11954/TT-12115]fix location of contextVariables (#6285) (#6304)\n\n### **User description**\r\n[TT-11954/TT-12115]fix location of contextVariables (#6285)\r\n\r\n### **User description**\r\n<!-- Provide a general summary of your changes in the Title above -->\r\n\r\n## Description\r\n\r\nfix context variables location\r\n## Related Issue\r\nParent: https://tyktech.atlassian.net/browse/TT-11954\r\nSubtask: https://tyktech.atlassian.net/browse/TT-12115\r\n\r\n## Motivation and Context\r\n\r\n<!-- Why is this change required? What problem does it solve? -->\r\n\r\n## How This Has Been Tested\r\n\r\n<!-- Please describe in detail how you tested your changes -->\r\n<!-- Include details of your testing environment, and the tests -->\r\n<!-- you ran to see how your change affects other areas of the code,\r\netc. -->\r\n<!-- This information is helpful for reviewers and QA. -->\r\n\r\n## Screenshots (if appropriate)\r\n\r\n## Types of changes\r\n\r\n<!-- What types of changes does your code introduce? Put an `x` in all\r\nthe boxes that apply: -->\r\n\r\n- [x] Bug fix (non-breaking change which fixes an issue)\r\n- [ ] New feature (non-breaking change which adds functionality)\r\n- [ ] Breaking change (fix or feature that would cause existing\r\nfunctionality to change)\r\n- [ ] Refactoring or add test (improvements in base code or adds test\r\ncoverage to functionality)\r\n\r\n## Checklist\r\n\r\n<!-- Go over all the following points, and put an `x` in all the boxes\r\nthat apply -->\r\n<!-- If there are no documentation updates required, mark the item as\r\nchecked. -->\r\n<!-- Raise up any additional concerns not covered by the checklist. -->\r\n\r\n- [ ] I ensured that the documentation is up to date\r\n- [ ] I explained why this PR updates go.mod in detail with reasoning\r\nwhy it's required\r\n- [ ] I would like a code coverage CI quality gate exception and have\r\nexplained why\r\n\r\n\r\n___\r\n\r\n### **PR Type**\r\nBug fix, Enhancement\r\n\r\n\r\n___\r\n\r\n### **Description**\r\n- Enhanced handling of `ContextVariables` in middleware to prevent their\r\nloss when global settings are omitted.\r\n- Added and updated tests for the new handling methods of\r\n`ContextVariables`.\r\n- Modified OAS migration logic to not enable `ContextVariables` by\r\ndefault, providing more control.\r\n- Implemented a new method `SetContextVariablesIfEmpty` to conditionally\r\nset `ContextVariables` if they are not already set.\r\n- Updated API creation logic to utilize the new method for setting\r\n`ContextVariables`, ensuring consistency and control.\r\n\r\n\r\n___\r\n\r\n\r\n\r\n### **Changes walkthrough** 📝\r\n<table><thead><tr><th></th><th align=\"left\">Relevant\r\nfiles</th></tr></thead><tbody><tr><td><strong>Enhancement\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>middleware.go</strong><dd><code>Enhance handling and\r\nlifecycle of ContextVariables in middleware</code></dd></summary>\r\n<hr>\r\n\r\napidef/oas/middleware.go\r\n<li>Added handling to preserve <code>ContextVariables</code> when\r\n<code>Global</code> middleware <br>settings are omitted.<br> <li>\r\nImplemented <code>Fill</code> and <code>ExtractTo</code> methods for\r\n<code>ContextVariables</code> to manage <br>their lifecycle in API\r\ndefinitions.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-992ec7c28d25fd54f6491d295389757705cd114bc869a35cba50d42e548cdc6e\">+46/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>oas.go</strong><dd><code>Modify default\r\nContextVariables behavior on OAS migration</code></dd></summary>\r\n<hr>\r\n\r\napidef/oas/oas.go\r\n<li>Removed default enabling of <code>ContextVariables</code> during OAS\r\nmigration to <br>allow more controlled management.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-80279b1d59499a41a77ff7a16a6e2c9b9b785a4fd1326c351da6884c867658d7\">+0/-1</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>root.go</strong><dd><code>Implement conditional setting\r\nof ContextVariables</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/root.go\r\n<li>Added <code>SetContextVariablesIfEmpty</code> to ensure\r\n<code>ContextVariables</code> are set <br>only if they are not already\r\nconfigured.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-9c56b2bdb992e0a7db76809d4c516e1cd61c9486c7f0437b344c0032476af80f\">+23/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>api.go</strong><dd><code>Adjust API creation logic to\r\nuse new ContextVariables setting method</code></dd></summary>\r\n<hr>\r\n\r\ngateway/api.go\r\n<li>Adjusted logic to use <code>SetContextVariablesIfEmpty</code> for\r\nsetting context <br>variables during API creation.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-644cda3aeb4ac7f325359e85fcddb810f100dd5e6fa480b0d9f9363a743c4e05\">+3/-3</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n</table></td></tr><tr><td><strong>Tests\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>middleware_test.go</strong><dd><code>Add tests for\r\nContextVariables handling in middleware</code>&nbsp; &nbsp; &nbsp;\r\n&nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/middleware_test.go\r\n<li>Added tests for <code>ContextVariables</code> to ensure correct\r\nbehavior of <code>Fill</code> <br>and <code>ExtractTo</code>\r\nmethods.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-0af31cb29ae298a6ac3e402b283ab364a6fd793fd04f253ef7c4983234c17bef\">+71/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>root_test.go</strong><dd><code>Test new conditional\r\nContextVariables setting method</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n</dd></summary>\r\n<hr>\r\n\r\napidef/oas/root_test.go\r\n<li>Added extensive testing for the new\r\n<code>SetContextVariablesIfEmpty</code> method <br>to verify its\r\nconditional logic.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-4d144465b7fabaf2db8915de1ce3b6ff8c91a93c62d614c38fa38bdae28e23a2\">+124/-0</a>&nbsp;\r\n</td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>api_test.go</strong><dd><code>Update API tests to\r\naccommodate new ContextVariables logic</code></dd></summary>\r\n<hr>\r\n\r\ngateway/api_test.go\r\n<li>Updated tests to reflect new handling of\r\n<code>ContextVariables</code> in API <br>creation and migration\r\nscenarios.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-10b4a3d7bdd8d98e48b288d27fd46d9ee436617806c46913fdf7942c0e4a992e\">+4/-4</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n</table></td></tr></tr></tbody></table>\r\n\r\n___\r\n\r\n> 💡 **PR-Agent usage**:\r\n>Comment `/help` on the PR to get a list of all available PR-Agent tools\r\nand their descriptions\r\n\r\n\r\n___\r\n\r\n### **PR Type**\r\nBug fix, Enhancement\r\n\r\n\r\n___\r\n\r\n### **Description**\r\n- Added `enableContextVariablesIfEmpty` method to ensure context\r\nvariables are enabled during import.\r\n- Added and updated tests to verify context variables are enabled by\r\ndefault.\r\n- Added `ContextVariables` field to `Global` struct and implemented\r\nmethods to handle them.\r\n- Removed redundant code and assertions related to context variables\r\nenabling.\r\n\r\n\r\n___\r\n\r\n\r\n\r\n### **Changes walkthrough** 📝\r\n<table><thead><tr><th></th><th align=\"left\">Relevant\r\nfiles</th></tr></thead><tbody><tr><td><strong>Enhancement\r\n</strong></td><td><details><summary>3 files</summary><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>default.go</strong><dd><code>Ensure context variables\r\nare enabled during import.</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/default.go\r\n<li>Added <code>enableContextVariablesIfEmpty</code> method call to\r\nensure context <br>variables are enabled during import.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6304/files#diff-83c3a85bdd05785178ee519b05b1fe2008435dc4ae9448d72b080b5f67c491ad\">+1/-0</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>middleware.go</strong><dd><code>Add context variables\r\nhandling in middleware.</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/middleware.go\r\n<li>Added <code>ContextVariables</code> field to <code>Global</code>\r\nstruct.<br> <li> Implemented methods to fill and extract context\r\nvariables.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6304/files#diff-992ec7c28d25fd54f6491d295389757705cd114bc869a35cba50d42e548cdc6e\">+46/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>root.go</strong><dd><code>Add method to enable context\r\nvariables if not set.</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/root.go\r\n- Added `enableContextVariablesIfEmpty` method to `XTykAPIGateway`.\r\n\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6304/files#diff-9c56b2bdb992e0a7db76809d4c516e1cd61c9486c7f0437b344c0032476af80f\">+18/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n</table></details></td></tr><tr><td><strong>Tests\r\n</strong></td><td><details><summary>5 files</summary><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>default_test.go</strong><dd><code>Add tests for default\r\ncontext variables enabling.</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/default_test.go\r\n- Added tests to verify context variables are enabled by default.\r\n\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6304/files#diff-ab6848f71731083885a9d7d7970faa68a6783a98477c78413ae3979cb5add7db\">+66/-3</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>middleware_test.go</strong><dd><code>Add tests for\r\nmiddleware context variables handling.</code>&nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/middleware_test.go\r\n- Added tests for context variables handling in middleware.\r\n\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6304/files#diff-0af31cb29ae298a6ac3e402b283ab364a6fd793fd04f253ef7c4983234c17bef\">+71/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>oas_test.go</strong><dd><code>Update tests to reflect\r\nremoval of redundant context variables\r\n</code><br><code>enabling.</code></dd></summary>\r\n<hr>\r\n\r\napidef/oas/oas_test.go\r\n<li>Removed assertions related to redundant context variables\r\nenabling.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6304/files#diff-74029ee88132d30d6478c96a35f8bb2200e0c8e6f42f2c9b147dc6bb7ce74644\">+0/-5</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>root_test.go</strong><dd><code>Add tests for enabling\r\ncontext variables if not set.</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n</dd></summary>\r\n<hr>\r\n\r\napidef/oas/root_test.go\r\n- Added tests for `enableContextVariablesIfEmpty` method.\r\n\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6304/files#diff-4d144465b7fabaf2db8915de1ce3b6ff8c91a93c62d614c38fa38bdae28e23a2\">+94/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>api_test.go</strong><dd><code>Update tests for context\r\nvariables enabling logic.</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\ngateway/api_test.go\r\n<li>Updated tests to reflect changes in context variables enabling\r\nlogic.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6304/files#diff-10b4a3d7bdd8d98e48b288d27fd46d9ee436617806c46913fdf7942c0e4a992e\">+1/-20</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n</table></details></td></tr><tr><td><strong>Bug fix\r\n</strong></td><td><details><summary>3 files</summary><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>oas.go</strong><dd><code>Remove redundant context\r\nvariables enabling.</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/oas.go\r\n<li>Removed redundant enabling of context variables in\r\n<br><code>newOASFromClassicAPIDefinition</code>.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6304/files#diff-80279b1d59499a41a77ff7a16a6e2c9b9b785a4fd1326c351da6884c867658d7\">+0/-1</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>api.go</strong><dd><code>Remove redundant context\r\nvariables enabling code.</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\ngateway/api.go\r\n- Removed redundant code for enabling context variables by default.\r\n\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6304/files#diff-644cda3aeb4ac7f325359e85fcddb810f100dd5e6fa480b0d9f9363a743c4e05\">+0/-5</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>x-tyk-api-gateway.json</strong><dd><code>Correct\r\nplacement of context variables definition.</code>&nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; &nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/schema/x-tyk-api-gateway.json\r\n- Moved `contextVariables` definition to the correct section.\r\n\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6304/files#diff-78828969c0c04cc1a776dfc93a8bad3c499a8c83e6169f83e96d090bed3e7dd0\">+3/-3</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n</table></details></td></tr></tr></tbody></table>\r\n\r\n___\r\n\r\n> 💡 **PR-Agent usage**:\r\n>Comment `/help` on the PR to get a list of all available PR-Agent tools\r\nand their descriptions\r\n\r\n---------\r\n\r\nCo-authored-by: Jeffy Mathew <jeffy.mathew100@gmail.com>\r\nCo-authored-by: Tit Petric <tit@tyk.io>","shortMessageHtmlLink":"Merging to release-5.3: [TT-11954/TT-12115]fix location of contextVar…"}},{"before":"6bc052808a9135e103b39d055005cabfa72ad481","after":"90653b2b40368b836dd1ef1650c10a58a838b0a4","ref":"refs/heads/merge/release-5.3/63963725d1b802d03841651f290d1cbd11f398e9","pushedAt":"2024-05-23T07:57:00.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"},"commit":{"message":"Remove more old code","shortMessageHtmlLink":"Remove more old code"}},{"before":"465b59d6660e6c759022d3b14147d6a8d092630e","after":"6bc052808a9135e103b39d055005cabfa72ad481","ref":"refs/heads/merge/release-5.3/63963725d1b802d03841651f290d1cbd11f398e9","pushedAt":"2024-05-23T07:34:24.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"},"commit":{"message":"Resolve conflicts","shortMessageHtmlLink":"Resolve conflicts"}},{"before":null,"after":"465b59d6660e6c759022d3b14147d6a8d092630e","ref":"refs/heads/merge/release-5.3/63963725d1b802d03841651f290d1cbd11f398e9","pushedAt":"2024-05-23T07:29:26.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"tykbot[bot]","name":null,"path":"/apps/tykbot","primaryAvatarUrl":"https://avatars.githubusercontent.com/in/35290?s=80&v=4"},"commit":{"message":"[TT-11954/TT-12115]fix location of contextVariables (#6285)\n\n<!-- Provide a general summary of your changes in the Title above -->\n\nfix context variables location\nParent: https://tyktech.atlassian.net/browse/TT-11954\nSubtask: https://tyktech.atlassian.net/browse/TT-12115\n\n<!-- Why is this change required? What problem does it solve? -->\n\n<!-- Please describe in detail how you tested your changes -->\n<!-- Include details of your testing environment, and the tests -->\n<!-- you ran to see how your change affects other areas of the code,\netc. -->\n<!-- This information is helpful for reviewers and QA. -->\n\n<!-- What types of changes does your code introduce? Put an `x` in all\nthe boxes that apply: -->\n\n- [x] Bug fix (non-breaking change which fixes an issue)\n- [ ] New feature (non-breaking change which adds functionality)\n- [ ] Breaking change (fix or feature that would cause existing\nfunctionality to change)\n- [ ] Refactoring or add test (improvements in base code or adds test\ncoverage to functionality)\n\n<!-- Go over all the following points, and put an `x` in all the boxes\nthat apply -->\n<!-- If there are no documentation updates required, mark the item as\nchecked. -->\n<!-- Raise up any additional concerns not covered by the checklist. -->\n\n- [ ] I ensured that the documentation is up to date\n- [ ] I explained why this PR updates go.mod in detail with reasoning\nwhy it's required\n- [ ] I would like a code coverage CI quality gate exception and have\nexplained why\n\n___\n\nBug fix, Enhancement\n\n___\n\n- Enhanced handling of `ContextVariables` in middleware to prevent their\nloss when global settings are omitted.\n- Added and updated tests for the new handling methods of\n`ContextVariables`.\n- Modified OAS migration logic to not enable `ContextVariables` by\ndefault, providing more control.\n- Implemented a new method `SetContextVariablesIfEmpty` to conditionally\nset `ContextVariables` if they are not already set.\n- Updated API creation logic to utilize the new method for setting\n`ContextVariables`, ensuring consistency and control.\n\n___\n\n<table><thead><tr><th></th><th align=\"left\">Relevant\nfiles</th></tr></thead><tbody><tr><td><strong>Enhancement\n</strong></td><td><table>\n<tr>\n  <td>\n    <details>\n<summary><strong>middleware.go</strong><dd><code>Enhance handling and\nlifecycle of ContextVariables in middleware</code></dd></summary>\n<hr>\n\napidef/oas/middleware.go\n<li>Added handling to preserve <code>ContextVariables</code> when\n<code>Global</code> middleware <br>settings are omitted.<br> <li>\nImplemented <code>Fill</code> and <code>ExtractTo</code> methods for\n<code>ContextVariables</code> to manage <br>their lifecycle in API\ndefinitions.<br>\n\n</details>\n\n  </td>\n<td><a\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-992ec7c28d25fd54f6491d295389757705cd114bc869a35cba50d42e548cdc6e\">+46/-0</a>&nbsp;\n&nbsp; </td>\n</tr>\n\n<tr>\n  <td>\n    <details>\n<summary><strong>oas.go</strong><dd><code>Modify default\nContextVariables behavior on OAS migration</code></dd></summary>\n<hr>\n\napidef/oas/oas.go\n<li>Removed default enabling of <code>ContextVariables</code> during OAS\nmigration to <br>allow more controlled management.<br>\n\n</details>\n\n  </td>\n<td><a\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-80279b1d59499a41a77ff7a16a6e2c9b9b785a4fd1326c351da6884c867658d7\">+0/-1</a>&nbsp;\n&nbsp; &nbsp; </td>\n</tr>\n\n<tr>\n  <td>\n    <details>\n<summary><strong>root.go</strong><dd><code>Implement conditional setting\nof ContextVariables</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\n&nbsp; &nbsp; </dd></summary>\n<hr>\n\napidef/oas/root.go\n<li>Added <code>SetContextVariablesIfEmpty</code> to ensure\n<code>ContextVariables</code> are set <br>only if they are not already\nconfigured.<br>\n\n</details>\n\n  </td>\n<td><a\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-9c56b2bdb992e0a7db76809d4c516e1cd61c9486c7f0437b344c0032476af80f\">+23/-0</a>&nbsp;\n&nbsp; </td>\n</tr>\n\n<tr>\n  <td>\n    <details>\n<summary><strong>api.go</strong><dd><code>Adjust API creation logic to\nuse new ContextVariables setting method</code></dd></summary>\n<hr>\n\ngateway/api.go\n<li>Adjusted logic to use <code>SetContextVariablesIfEmpty</code> for\nsetting context <br>variables during API creation.<br>\n\n</details>\n\n  </td>\n<td><a\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-644cda3aeb4ac7f325359e85fcddb810f100dd5e6fa480b0d9f9363a743c4e05\">+3/-3</a>&nbsp;\n&nbsp; &nbsp; </td>\n</tr>\n</table></td></tr><tr><td><strong>Tests\n</strong></td><td><table>\n<tr>\n  <td>\n    <details>\n<summary><strong>middleware_test.go</strong><dd><code>Add tests for\nContextVariables handling in middleware</code>&nbsp; &nbsp; &nbsp;\n&nbsp; </dd></summary>\n<hr>\n\napidef/oas/middleware_test.go\n<li>Added tests for <code>ContextVariables</code> to ensure correct\nbehavior of <code>Fill</code> <br>and <code>ExtractTo</code>\nmethods.<br>\n\n</details>\n\n  </td>\n<td><a\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-0af31cb29ae298a6ac3e402b283ab364a6fd793fd04f253ef7c4983234c17bef\">+71/-0</a>&nbsp;\n&nbsp; </td>\n</tr>\n\n<tr>\n  <td>\n    <details>\n<summary><strong>root_test.go</strong><dd><code>Test new conditional\nContextVariables setting method</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\n</dd></summary>\n<hr>\n\napidef/oas/root_test.go\n<li>Added extensive testing for the new\n<code>SetContextVariablesIfEmpty</code> method <br>to verify its\nconditional logic.<br>\n\n</details>\n\n  </td>\n<td><a\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-4d144465b7fabaf2db8915de1ce3b6ff8c91a93c62d614c38fa38bdae28e23a2\">+124/-0</a>&nbsp;\n</td>\n</tr>\n\n<tr>\n  <td>\n    <details>\n<summary><strong>api_test.go</strong><dd><code>Update API tests to\naccommodate new ContextVariables logic</code></dd></summary>\n<hr>\n\ngateway/api_test.go\n<li>Updated tests to reflect new handling of\n<code>ContextVariables</code> in API <br>creation and migration\nscenarios.<br>\n\n</details>\n\n  </td>\n<td><a\nhref=\"https://github.com/TykTechnologies/tyk/pull/6285/files#diff-10b4a3d7bdd8d98e48b288d27fd46d9ee436617806c46913fdf7942c0e4a992e\">+4/-4</a>&nbsp;\n&nbsp; &nbsp; </td>\n</tr>\n</table></td></tr></tr></tbody></table>\n\n___\n\n> 💡 **PR-Agent usage**:\n>Comment `/help` on the PR to get a list of all available PR-Agent tools\nand their descriptions\n\n(cherry picked from commit 63963725d1b802d03841651f290d1cbd11f398e9)","shortMessageHtmlLink":"[TT-11954/TT-12115]fix location of contextVariables (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"2299741811\" data-permission-text=\"Title is private\" data-url=\"https://github.com/TykTechnologies/tyk/issues/6285\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/TykTechnologies/tyk/pull/6285/hovercard\" href=\"https://github.com/TykTechnologies/tyk/pull/6285\">#6285</a>)"}},{"before":"e10f3c239532eacb30cdb25c0b0b3bbf95cd7e18","after":null,"ref":"refs/heads/merge/release-5.3/6b7c9ad54c33314499192f0feba7b8a717b40a01","pushedAt":"2024-05-23T07:23:10.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"}},{"before":"05ec0f52b31bda6f0d6be6f9393e06722cb5a222","after":"0038992886cd297e7aabeb473aa25d6e180557bb","ref":"refs/heads/release-5.3","pushedAt":"2024-05-23T07:23:08.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"},"commit":{"message":"[TT-11954/TT-12155] add x-tyk-api-gateway.servers.contextVariables.en… (#6302)\n\n### **User description**\r\n…abled (#6281)\r\n\r\n<!-- Provide a general summary of your changes in the Title above -->\r\n\r\n- add x-tyk-api-gateway.servers.contextVariables.enabled\r\n- enable context variables by default for OAS APIs if not explicitly\r\nconfigured otherwise.\r\n\r\n<!-- Describe your changes in detail -->\r\nParent: https://tyktech.atlassian.net/browse/TT-11954 Subtask:\r\nhttps://tyktech.atlassian.net/browse/TT-12155 <!-- This project only\r\naccepts pull requests related to open issues. --> <!-- If suggesting a\r\nnew feature or change, please discuss it in an issue first. -->\r\n<!-- If fixing a bug, there should be an issue describing it with steps\r\nto reproduce. -->\r\n<!-- OSS: Please link to the issue here. Tyk: please create/link the\r\nJIRA ticket. -->\r\n\r\n<!-- Why is this change required? What problem does it solve? -->\r\n\r\n<!-- Please describe in detail how you tested your changes --> <!--\r\nInclude details of your testing environment, and the tests --> <!-- you\r\nran to see how your change affects other areas of the code, etc. -->\r\n<!-- This information is helpful for reviewers and QA. -->\r\n\r\n<!-- What types of changes does your code introduce? Put an `x` in all\r\nthe boxes that apply: -->\r\n\r\n- [ ] Bug fix (non-breaking change which fixes an issue)\r\n- [ ] New feature (non-breaking change which adds functionality)\r\n- [ ] Breaking change (fix or feature that would cause existing\r\nfunctionality to change)\r\n- [ ] Refactoring or add test (improvements in base code or adds test\r\ncoverage to functionality)\r\n\r\n<!-- Go over all the following points, and put an `x` in all the boxes\r\nthat apply -->\r\n<!-- If there are no documentation updates required, mark the item as\r\nchecked. -->\r\n<!-- Raise up any additional concerns not covered by the checklist. -->\r\n\r\n- [ ] I ensured that the documentation is up to date\r\n- [ ] I explained why this PR updates go.mod in detail with reasoning\r\nwhy it's required\r\n- [ ] I would like a code coverage CI quality gate exception and have\r\nexplained why\r\n\r\n___\r\n\r\nenhancement\r\n\r\n___\r\n\r\n- Added handling and configuration for context variables in the OAS\r\nserver structure.\r\n- Removed hardcoded enabling of context variables, now configurable via\r\nJSON.\r\n- Updated tests to reflect new logic and added new tests for context\r\nvariable configuration.\r\n- Updated JSON schema to include new `ContextVariables` definition.\r\n\r\n___\r\n\r\n<table><thead><tr><th></th><th align=\"left\">Relevant\r\nfiles</th></tr></thead><tbody><tr><td><strong>Tests\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>oas_test.go</strong><dd><code>Update OAS Test to\r\nReflect Context Variable Changes</code>&nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/oas_test.go\r\n- Removed assertion for `EnableContextVars` in `TestOAS`.\r\n\r\n</details>\r\n\r\n  </td> <td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6281/files#diff-74029ee88132d30d6478c96a35f8bb2200e0c8e6f42f2c9b147dc6bb7ce74644\">+0/-1</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>\r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>server_test.go</strong><dd><code>Add Tests for Context\r\nVariables in Server Struct</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; &nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/server_test.go\r\n- Added tests for `ContextVariables` handling in `Server` struct.\r\n\r\n</details>\r\n\r\n  </td> <td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6281/files#diff-a194795077946243b2b6d04b30be360dbafff8e5eb03e6212081e45d1c72573f\">+71/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>\r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>api_test.go</strong><dd><code>Test Default Enabling of\r\nContext Variables in OAS APIs</code>&nbsp; &nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\ngateway/api_test.go\r\n<li>Added tests to ensure context variables are enabled by default and\r\n<br>respect explicit configuration.<br>\r\n\r\n</details>\r\n\r\n  </td>\r\n<td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6281/files#diff-10b4a3d7bdd8d98e48b288d27fd46d9ee436617806c46913fdf7942c0e4a992e\">+21/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>\r\n</table></td></tr><tr><td><strong>Enhancement\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>root.go</strong><dd><code>Remove Hardcoded Context\r\nVariable Enabling in Root Extraction</code></dd></summary> <hr>\r\n\r\napidef/oas/root.go\r\n- Removed hardcoded enabling of `EnableContextVars`.\r\n\r\n</details>\r\n\r\n  </td> <td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6281/files#diff-9c56b2bdb992e0a7db76809d4c516e1cd61c9486c7f0437b344c0032476af80f\">+0/-3</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>\r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>server.go</strong><dd><code>Integrate Context Variables\r\nHandling in Server Struct</code>&nbsp; &nbsp; &nbsp; &nbsp;\r\n</dd></summary>\r\n<hr>\r\n\r\napidef/oas/server.go\r\n<li>Added handling for <code>ContextVariables</code> in\r\n<code>Server</code> struct.<br> <li> Implemented <code>Fill</code> and\r\n<code>ExtractTo</code> methods for <code>ContextVariables</code>.<br>\r\n\r\n</details>\r\n\r\n  </td>\r\n<td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6281/files#diff-21857c42e8659f7980014e277c3c758703f29e9e5c0c40553f2584cddb870808\">+41/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>\r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>api.go</strong><dd><code>Default Enable Context\r\nVariables for OAS APIs</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; </dd></summary> <hr>\r\n\r\ngateway/api.go\r\n<li>Added default enabling of context variables for OAS APIs if not\r\n<br>explicitly set.<br>\r\n\r\n</details>\r\n\r\n  </td>\r\n<td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6281/files#diff-644cda3aeb4ac7f325359e85fcddb810f100dd5e6fa480b0d9f9363a743c4e05\">+5/-0</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>\r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>x-tyk-api-gateway.json</strong><dd><code>Add JSON\r\nSchema for Context Variables</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; </dd></summary> <hr>\r\n\r\napidef/oas/schema/x-tyk-api-gateway.json\r\n- Added schema definition for `ContextVariables`.\r\n\r\n</details>\r\n\r\n  </td> <td><a\r\n\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6281/files#diff-78828969c0c04cc1a776dfc93a8bad3c499a8c83e6169f83e96d090bed3e7dd0\">+14/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>\r\n</table></td></tr></tr></tbody></table>\r\n\r\n___\r\n\r\n> 💡 **PR-Agent usage**:\r\n>Comment `/help` on the PR to get a list of all available PR-Agent tools\r\nand their descriptions\r\n\r\n<!-- Provide a general summary of your changes in the Title above -->\r\n\r\n## Description\r\n\r\n<!-- Describe your changes in detail -->\r\n\r\n## Related Issue\r\n\r\n<!-- This project only accepts pull requests related to open issues. -->\r\n<!-- If suggesting a new feature or change, please discuss it in an\r\nissue first. -->\r\n<!-- If fixing a bug, there should be an issue describing it with steps\r\nto reproduce. -->\r\n<!-- OSS: Please link to the issue here. Tyk: please create/link the\r\nJIRA ticket. -->\r\n\r\n## Motivation and Context\r\n\r\n<!-- Why is this change required? What problem does it solve? -->\r\n\r\n## How This Has Been Tested\r\n\r\n<!-- Please describe in detail how you tested your changes -->\r\n<!-- Include details of your testing environment, and the tests -->\r\n<!-- you ran to see how your change affects other areas of the code,\r\netc. -->\r\n<!-- This information is helpful for reviewers and QA. -->\r\n\r\n## Screenshots (if appropriate)\r\n\r\n## Types of changes\r\n\r\n<!-- What types of changes does your code introduce? Put an `x` in all\r\nthe boxes that apply: -->\r\n\r\n- [ ] Bug fix (non-breaking change which fixes an issue)\r\n- [ ] New feature (non-breaking change which adds functionality)\r\n- [ ] Breaking change (fix or feature that would cause existing\r\nfunctionality to change)\r\n- [ ] Refactoring or add test (improvements in base code or adds test\r\ncoverage to functionality)\r\n\r\n## Checklist\r\n\r\n<!-- Go over all the following points, and put an `x` in all the boxes\r\nthat apply -->\r\n<!-- If there are no documentation updates required, mark the item as\r\nchecked. -->\r\n<!-- Raise up any additional concerns not covered by the checklist. -->\r\n\r\n- [ ] I ensured that the documentation is up to date\r\n- [ ] I explained why this PR updates go.mod in detail with reasoning\r\nwhy it's required\r\n- [ ] I would like a code coverage CI quality gate exception and have\r\nexplained why\r\n\r\n\r\n___\r\n\r\n### **PR Type**\r\nEnhancement, Tests\r\n\r\n\r\n___\r\n\r\n### **Description**\r\n- Added `ContextVariables` struct and related methods to manage context\r\nvariables in OAS APIs.\r\n- Updated `Fill` and `ExtractTo` methods in `Server` to handle\r\n`ContextVariables`.\r\n- Enabled context variables by default for OAS APIs if not explicitly\r\nset to false.\r\n- Added tests to ensure context variables are enabled by default and can\r\nbe explicitly disabled.\r\n- Updated schema to include `contextVariables` and defined\r\n`X-Tyk-ContextVariables`.\r\n\r\n\r\n___\r\n\r\n\r\n\r\n### **Changes walkthrough** 📝\r\n<table><thead><tr><th></th><th align=\"left\">Relevant\r\nfiles</th></tr></thead><tbody><tr><td><strong>Tests\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>oas_test.go</strong><dd><code>Remove assertion for\r\n`EnableContextVars` in OAS test.</code>&nbsp; &nbsp; &nbsp; &nbsp;\r\n</dd></summary>\r\n<hr>\r\n\r\napidef/oas/oas_test.go\r\n- Removed assertion for `EnableContextVars` in `TestOAS`.\r\n\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6302/files#diff-74029ee88132d30d6478c96a35f8bb2200e0c8e6f42f2c9b147dc6bb7ce74644\">+0/-1</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>server_test.go</strong><dd><code>Add tests for\r\nContextVariables in Server.</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; </dd></summary>\r\n<hr>\r\n\r\napidef/oas/server_test.go\r\n- Added tests for `ContextVariables` in `Server`.\r\n\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6302/files#diff-a194795077946243b2b6d04b30be360dbafff8e5eb03e6212081e45d1c72573f\">+71/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>api_test.go</strong><dd><code>Add tests for default and\r\nexplicit enabling of context variables.</code></dd></summary>\r\n<hr>\r\n\r\ngateway/api_test.go\r\n<li>Added tests to ensure context variables are enabled by default.<br>\r\n<li> Added tests to ensure context variables can be explicitly\r\ndisabled.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6302/files#diff-10b4a3d7bdd8d98e48b288d27fd46d9ee436617806c46913fdf7942c0e4a992e\">+21/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n</table></td></tr><tr><td><strong>Enhancement\r\n</strong></td><td><table>\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>root.go</strong><dd><code>Remove always enabling\r\ncontext variables in ExtractTo method.</code></dd></summary>\r\n<hr>\r\n\r\napidef/oas/root.go\r\n- Removed always enabling `EnableContextVars` in `ExtractTo` method.\r\n\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6302/files#diff-9c56b2bdb992e0a7db76809d4c516e1cd61c9486c7f0437b344c0032476af80f\">+0/-3</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>server.go</strong><dd><code>Add ContextVariables struct\r\nand update Fill and ExtractTo methods.</code></dd></summary>\r\n<hr>\r\n\r\napidef/oas/server.go\r\n<li>Added <code>ContextVariables</code> struct and related methods.<br>\r\n<li> Updated <code>Fill</code> and <code>ExtractTo</code> methods to\r\nhandle <code>ContextVariables</code>.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6302/files#diff-21857c42e8659f7980014e277c3c758703f29e9e5c0c40553f2584cddb870808\">+41/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>api.go</strong><dd><code>Enable context variables by\r\ndefault for OAS APIs.</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;\r\n&nbsp; &nbsp; </dd></summary>\r\n<hr>\r\n\r\ngateway/api.go\r\n<li>Enabled context variables by default for OAS APIs if not explicitly\r\n<br>set.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6302/files#diff-644cda3aeb4ac7f325359e85fcddb810f100dd5e6fa480b0d9f9363a743c4e05\">+5/-0</a>&nbsp;\r\n&nbsp; &nbsp; </td>\r\n</tr>                    \r\n\r\n<tr>\r\n  <td>\r\n    <details>\r\n<summary><strong>x-tyk-api-gateway.json</strong><dd><code>Add\r\ncontextVariables to schema and define\r\nX-Tyk-ContextVariables.</code></dd></summary>\r\n<hr>\r\n\r\napidef/oas/schema/x-tyk-api-gateway.json\r\n<li>Added <code>contextVariables</code> to the schema.<br> <li> Added\r\n<code>X-Tyk-ContextVariables</code> definition.<br>\r\n\r\n\r\n</details>\r\n    \r\n\r\n  </td>\r\n<td><a\r\nhref=\"https://github.com/TykTechnologies/tyk/pull/6302/files#diff-78828969c0c04cc1a776dfc93a8bad3c499a8c83e6169f83e96d090bed3e7dd0\">+14/-0</a>&nbsp;\r\n&nbsp; </td>\r\n</tr>                    \r\n</table></td></tr></tr></tbody></table>\r\n\r\n___\r\n\r\n> 💡 **PR-Agent usage**:\r\n>Comment `/help` on the PR to get a list of all available PR-Agent tools\r\nand their descriptions\r\n\r\n---------\r\n\r\nCo-authored-by: Jeffy Mathew <jeffy.mathew100@gmail.com>\r\nCo-authored-by: Tit Petric <tit@tyk.io>","shortMessageHtmlLink":"[TT-11954/TT-12155] add x-tyk-api-gateway.servers.contextVariables.en… ("}},{"before":null,"after":"fe24bf0824f17f74906d14c707e96fcf9e51f952","ref":"refs/heads/fix/event-handler-webhooks-error-handling","pushedAt":"2024-05-23T07:18:42.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"titpetric","name":"Tit Petric","path":"/titpetric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/233360?s=80&v=4"},"commit":{"message":"Fix poor error handling in webhook event templates","shortMessageHtmlLink":"Fix poor error handling in webhook event templates"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAEUkVYvwA","startCursor":null,"endCursor":null}},"title":"Activity · TykTechnologies/tyk"}