Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TT-8959/TT-9611] Add timeout to endpoint level cache config #5368

Merged
merged 1 commit into from
Jul 28, 2023

Conversation

furkansenharputlu
Copy link
Contributor

This PR adds a new param timeout to advance_cache_config so that APIDefinition.CacheOptions.CacheTimeout is overriden by endpoint level extended_paths.advance_cache_config[x].timeout if endpoint level is greater than 0.

@furkansenharputlu furkansenharputlu force-pushed the feature/enpoint-level-cache-timeout branch from cace868 to 73d1720 Compare July 27, 2023 16:08
@furkansenharputlu furkansenharputlu changed the title [TT-8959] Add timeout to endpoint level cache config [TT-8959/TT-9611] Add timeout to endpoint level cache config Jul 27, 2023
@sonarcloud
Copy link

sonarcloud bot commented Jul 27, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@Tyk-ITS
Copy link
Contributor

Tyk-ITS commented Jul 27, 2023

API tests result: success
Branch used: refs/pull/5368/merge
Commit:
Triggered by: pull_request (@furkansenharputlu)
Execution page

@Tyk-ITS
Copy link
Contributor

Tyk-ITS commented Jul 27, 2023

API tests result: success
Branch used: refs/pull/5368/merge
Commit: 73d1720
Triggered by: pull_request (@furkansenharputlu)
Execution page

@Tyk-ITS
Copy link
Contributor

Tyk-ITS commented Jul 27, 2023

API tests result: success
Branch used: refs/pull/5368/merge
Commit: cace868
Triggered by: pull_request (@furkansenharputlu)
Execution page

@furkansenharputlu furkansenharputlu merged commit 3cbee71 into master Jul 28, 2023
16 of 18 checks passed
@furkansenharputlu furkansenharputlu deleted the feature/enpoint-level-cache-timeout branch July 28, 2023 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants