tyk
tyk copied to clipboard
[TT-10446]: added a custom multiplexer to be used for internal requests
Description
This fix adds a custom multiplexer in the reverse proxy to be used to handle internal routing. Why? Because currently, the request context propagation uses the first parent request's context, instead of subsequent requests. TT-10446
Related Issue
Motivation and Context
How This Has Been Tested
Screenshots (if appropriate)
Types of changes
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
- [ ] Refactoring or add test (improvements in base code or adds test coverage to functionality)
Checklist
- [ ] I ensured that the documentation is up to date
- [ ] I explained why this PR updates go.mod in detail with reasoning why it's required
- [ ] I would like a code coverage CI quality gate exception and have explained why
API Changes
--- prev.txt 2023-11-28 08:34:50.114478235 +0000
+++ current.txt 2023-11-28 08:34:46.722497298 +0000
@@ -6064,6 +6064,7 @@
// CacheOptions holds cache options required for cache writer middleware.
CacheOptions
OASDefinition
+ APIID
)
# Package: ./dlpython
Apply Sweep Rules to your PR?
- [ ] Apply: All new business logic should have corresponding unit tests.
- [ ] Apply: Refactor large functions to be more modular.
- [ ] Apply: Add docstrings to all functions and file headers.
SonarCloud Quality Gate failed.
1 Bug
0 Vulnerabilities
0 Security Hotspots
0 Code Smells
82.4% Coverage
0.0% Duplication
Catch issues before they fail your Quality Gate with our IDE extension
SonarLint
API tests result: success :white_check_mark: Branch used: refs/pull/5799/merge Commit: d3791a0b6427b2e933cf5a353e25d8e43f3e8dc9 Triggered by: pull_request (@kofoworola) Execution page
:boom: CI tests failed :see_no_evil:
git-state
diff --git a/internal/graphql/otel_graphql_engine_detailed.go b/internal/graphql/otel_graphql_engine_detailed.go
index f2793d6..7460350 100644
--- a/internal/graphql/otel_graphql_engine_detailed.go
+++ b/internal/graphql/otel_graphql_engine_detailed.go
@@ -2,9 +2,10 @@ package graphql
import (
"context"
- tykctx "github.com/TykTechnologies/tyk/ctx"
"sync"
+ tykctx "github.com/TykTechnologies/tyk/ctx"
+
"github.com/TykTechnologies/graphql-go-tools/pkg/ast"
"github.com/TykTechnologies/graphql-go-tools/pkg/engine/plan"
"github.com/TykTechnologies/graphql-go-tools/pkg/engine/resolve"
Please look at the run or in the Checks tab.
API tests result: success :white_check_mark: Branch used: refs/pull/5799/merge Commit: da8c7dddd1785cf14fdbeff5f0d4f67e35529977 Triggered by: pull_request (@kofoworola) Execution page
API tests result - postgres15-murmur64 env: success :white_check_mark: Branch used: refs/pull/5799/merge Commit: a0f309d876188fedbd55efb1b940edcaa5bb181b Triggered by: pull_request (@kofoworola) Execution page
API tests result - postgres15-sha256 env: success :white_check_mark: Branch used: refs/pull/5799/merge Commit: a0f309d876188fedbd55efb1b940edcaa5bb181b Triggered by: pull_request (@kofoworola) Execution page
API tests result - mongo44-sha256 env: success :white_check_mark: Branch used: refs/pull/5799/merge Commit: a0f309d876188fedbd55efb1b940edcaa5bb181b Triggered by: pull_request (@kofoworola) Execution page
API tests result - mongo44-murmur64 env: success :white_check_mark: Branch used: refs/pull/5799/merge Commit: a0f309d876188fedbd55efb1b940edcaa5bb181b Triggered by: pull_request (@kofoworola) Execution page
:boom: CI tests failed :see_no_evil:
git-state
diff --git a/internal/graphql/otel_graphql_engine_detailed.go b/internal/graphql/otel_graphql_engine_detailed.go
index f2793d6..7460350 100644
--- a/internal/graphql/otel_graphql_engine_detailed.go
+++ b/internal/graphql/otel_graphql_engine_detailed.go
@@ -2,9 +2,10 @@ package graphql
import (
"context"
- tykctx "github.com/TykTechnologies/tyk/ctx"
"sync"
+ tykctx "github.com/TykTechnologies/tyk/ctx"
+
"github.com/TykTechnologies/graphql-go-tools/pkg/ast"
"github.com/TykTechnologies/graphql-go-tools/pkg/engine/plan"
"github.com/TykTechnologies/graphql-go-tools/pkg/engine/resolve"
Please look at the run or in the Checks tab.
API tests result: success :white_check_mark: Branch used: refs/pull/5799/merge Commit: a0f309d876188fedbd55efb1b940edcaa5bb181b Triggered by: pull_request (@kofoworola) Execution page
Removed us as we have no input.