Skip to content

feat: Added changes to support CRUD oprn for graphql ratelimiting cos…#1937

Merged
shivaygupta-dotcom merged 29 commits intomainfrom
feature/decK-1909
Apr 23, 2026
Merged

feat: Added changes to support CRUD oprn for graphql ratelimiting cos…#1937
shivaygupta-dotcom merged 29 commits intomainfrom
feature/decK-1909

Conversation

@shivaygupta-dotcom
Copy link
Copy Markdown
Contributor

@shivaygupta-dotcom shivaygupta-dotcom commented Mar 14, 2026

GH issue: #1909
jira: https://konghq.atlassian.net/browse/FTI-7307

Summary:

  1. Added new custom entity type graphql_ratelimiting_cost_decorations.
  2. Added tests in sync, dump and apply command.

Supporting PRs:
GDR: Kong/go-database-reconciler#428
Go-kong:Kong/go-kong#608

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 15, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 32.57%. Comparing base (9b74bcb) to head (75a585f).

Files with missing lines Patch % Lines
tests/integration/test_utils.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1937      +/-   ##
==========================================
+ Coverage   32.53%   32.57%   +0.04%     
==========================================
  Files          77       77              
  Lines        8758     8758              
==========================================
+ Hits         2849     2853       +4     
+ Misses       5713     5710       -3     
+ Partials      196      195       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@shivaygupta-dotcom
Copy link
Copy Markdown
Contributor Author

fix for CI is already raised by @Prashansa-K in this PR
#1940
I will back-merge main once this is merged.

@gitguardian
Copy link
Copy Markdown

gitguardian Bot commented Mar 23, 2026

⚠️ GitGuardian has uncovered 15 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
27577964 Triggered Generic Password fb97172 tests/integration/testdata/dump/009-skip-defaults/enterprise/3.4/credentials.expected.yaml View secret
27577956 Triggered Generic Password fb97172 tests/integration/testdata/sync/047-basic-auth-skip-hash/expected-dump.yaml View secret
27577949 Triggered Generic High Entropy Secret a8d4747 tests/integration/testdata/dump/009-skip-defaults/enterprise/3.14+/plugin-partial-2.yaml View secret
30400066 Triggered Generic High Entropy Secret a8d4747 tests/integration/testdata/diff/004-no-diff-plugin/kong.yaml View secret
27577959 Triggered Generic Password a8d4747 tests/integration/testdata/dump/008-sanitizer/consumergroup-plugins313.expected.yaml View secret
27577938 Triggered Generic High Entropy Secret a8d4747 tests/integration/testdata/dump/009-skip-defaults/enterprise/3.14+/credentials.expected.yaml View secret
27577959 Triggered Generic Password a8d4747 tests/integration/testdata/dump/008-sanitizer/consumergroup-plugins314.expected.yaml View secret
27577943 Triggered Generic High Entropy Secret a8d4747 tests/integration/testdata/dump/009-skip-defaults/enterprise/3.14+/credentials.expected.yaml View secret
27577949 Triggered Generic High Entropy Secret a8d4747 tests/integration/testdata/dump/009-skip-defaults/enterprise/3.14+/plugin-partial-2.expected.yaml View secret
27577943 Triggered Generic High Entropy Secret a8d4747 tests/integration/testdata/dump/009-skip-defaults/enterprise/3.14+/credentials.yaml View secret
27577959 Triggered Generic Password a8d4747 tests/integration/testdata/dump/008-sanitizer/consumergroup-plugins314.expected.yaml View secret
27577938 Triggered Generic High Entropy Secret a8d4747 tests/integration/testdata/dump/009-skip-defaults/enterprise/3.14+/credentials.yaml View secret
27577951 Triggered Generic Password a8d4747 tests/integration/testdata/dump/009-skip-defaults/enterprise/3.14+/credentials.yaml View secret
29995049 Triggered Generic Password a8d4747 tests/integration/testdata/dump/009-skip-defaults/enterprise/3.14+/credentials.expected.yaml View secret
27577959 Triggered Generic Password a8d4747 tests/integration/testdata/dump/008-sanitizer/consumergroup-plugins313.expected.yaml View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secrets safely. Learn here the best practices.
  3. Revoke and rotate these secrets.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@shivaygupta-dotcom shivaygupta-dotcom marked this pull request as ready for review March 23, 2026 07:12
@shivaygupta-dotcom shivaygupta-dotcom removed the request for review from Prashansa-K March 27, 2026 02:53
Comment thread go.mod Outdated
Comment thread tests/integration/dump_test.go Outdated
Comment thread tests/integration/dump_test.go
Comment thread tests/integration/sync_test.go Outdated
Comment thread tests/integration/sync_test.go Outdated
Comment thread tests/integration/sync_test.go Outdated
Comment thread tests/integration/sync_test.go Outdated
@shivaygupta-dotcom shivaygupta-dotcom merged commit eb536e5 into main Apr 23, 2026
48 of 51 checks passed
@shivaygupta-dotcom shivaygupta-dotcom deleted the feature/decK-1909 branch April 23, 2026 07:28
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