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

chore: updates from main branch #1802

Merged
merged 44 commits into from
Sep 16, 2024
Merged

chore: updates from main branch #1802

merged 44 commits into from
Sep 16, 2024

Conversation

dadrus
Copy link
Owner

@dadrus dadrus commented Sep 16, 2024

No description provided.

dadrus and others added 30 commits April 17, 2024 14:50
Co-authored-by: Dimitrij Drus <[email protected]>
Co-authored-by: David van der Spek <[email protected]>

---

perf: O(log(n)) time complexity for lookup of rules

feat: Support for free and single (named) wildcards for request path matching and access of the captured values from the pipeline

feat: Support for backtracking while matching rules

feat: Multiple rules can be defined for the same path, e.g. to have separate rules for read and write requests

feat: Glob expressions are context aware and use `.` for host related expressions and `/` for path related ones as separators

refactor!: Rule matching configuration API redesigned

refactor!: Default rule rejects requests with encoded slashes in the path of the URL with `400 Bad Request`
…used with `metadata_endpoint` (#1658)

---
Co-authored-by: Martin Koppehel <[email protected]>
Co-authored-by: Dimitrij Drus <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: dadrus <[email protected]>
Co-authored-by: Martin Koppehel <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
fix: Fixed a nil pointer error in the Helm chart that occurred when a deployment was configured with custom annotations due to an incorrect reference in the deployment template (#1752).

fix: Updated the admission controller configuration in the Helm chart to align with the redesigned structure done in v0.12.0-alpha release of heimdall (#1752).

fix: Corrected the placement of namespace selector properties in the Helm chart's admission controller configuration (#1752).
dadrus and others added 14 commits September 13, 2024 11:53
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: dadrus <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…ation version (#1798)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link

codecov bot commented Sep 16, 2024

Codecov Report

Attention: Patch coverage is 92.13974% with 36 lines in your changes missing coverage. Please review.

Project coverage is 89.75%. Comparing base (05bfe3d) to head (2131d3b).
Report is 45 commits behind head on release.

Files with missing lines Patch % Lines
cmd/validate/ruleset.go 64.70% 6 Missing ⚠️
...s/endpoint/authstrategy/http_message_signatures.go 94.95% 6 Missing ⚠️
...nal/handler/envoyextauth/grpcv3/request_context.go 33.33% 4 Missing ⚠️
internal/handler/requestcontext/request_context.go 33.33% 4 Missing ⚠️
internal/rules/config/rule.go 85.71% 1 Missing and 2 partials ⚠️
internal/rules/default_execution_condition.go 0.00% 3 Missing ⚠️
...ules/endpoint/authstrategy/mapstructure_decoder.go 88.00% 2 Missing and 1 partial ⚠️
internal/otel/metrics/certificate/observer.go 94.87% 1 Missing and 1 partial ⚠️
...handler/middleware/grpc/otelmetrics/interceptor.go 75.00% 1 Missing ⚠️
internal/otel/metrics/module.go 0.00% 1 Missing ⚠️
... and 3 more
Additional details and impacted files
@@             Coverage Diff             @@
##           release    #1802      +/-   ##
===========================================
+ Coverage    89.29%   89.75%   +0.45%     
===========================================
  Files          270      271       +1     
  Lines         8876     9154     +278     
===========================================
+ Hits          7926     8216     +290     
+ Misses         701      697       -4     
+ Partials       249      241       -8     

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

@dadrus dadrus merged commit 31631c5 into release Sep 16, 2024
24 checks passed
@dadrus dadrus deleted the chore/from_main branch September 16, 2024 13:58
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.

2 participants