Skip to content

Commit

Permalink
UPDATE: address comments on PR
Browse files Browse the repository at this point in the history 
Signed-off-by: Jim Fitzpatrick <[email protected]>
  • Loading branch information
@Boomatang
Boomatang committed Oct 15, 2024
1 parent 2a30f66 commit 044e52b
Show file tree
Hide file tree
Showing 6 changed files with 1 addition and 136 deletions.
75 changes: 0 additions & 75 deletions api/v1beta2/route_selectors.go
View file

This file was deleted.

6 changes: 0 additions & 6 deletions api/v1beta3/authpolicy_types.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -34,19 +34,16 @@ type AuthSchemeSpec struct {
// Authentication configs.
// At least one config MUST evaluate to a valid identity object for the auth request to be successful.
// +optional
// +kubebuilder:validation:MaxProperties=10
Authentication map[string]AuthenticationSpec `json:"authentication,omitempty"`

// Metadata sources.
// Authorino fetches auth metadata as JSON from sources specified in this config.
// +optional
// +kubebuilder:validation:MaxProperties=10
Metadata map[string]MetadataSpec `json:"metadata,omitempty"`

// Authorization policies.
// All policies MUST evaluate to "allowed = true" for the auth request be successful.
// +optional
// +kubebuilder:validation:MaxProperties=10
Authorization map[string]AuthorizationSpec `json:"authorization,omitempty"`

// Response items.
Expand All @@ -57,7 +54,6 @@ type AuthSchemeSpec struct {
// Callback functions.
// Authorino sends callbacks at the end of the auth pipeline to the endpoints specified in this config.
// +optional
// +kubebuilder:validation:MaxProperties=10
Callbacks map[string]CallbackSpec `json:"callbacks,omitempty"`
}

Expand Down Expand Up @@ -101,13 +97,11 @@ type ResponseSpec struct {
type WrappedSuccessResponseSpec struct {
// Custom success response items wrapped as HTTP headers.
// For integration of Authorino via proxy, the proxy must use these settings to inject data in the request.
// +kubebuilder:validation:MaxProperties=10
Headers map[string]HeaderSuccessResponseSpec `json:"headers,omitempty"`

// Custom success response items wrapped as HTTP headers.
// For integration of Authorino via proxy, the proxy must use these settings to propagate dynamic metadata.
// See https://www.envoyproxy.io/docs/envoy/latest/configuration/advanced/well_known_dynamic_metadata
// +kubebuilder:validation:MaxProperties=10
DynamicMetadata map[string]SuccessResponseSpec `json:"dynamicMetadata,omitempty"`
}

Expand Down
2 changes: 1 addition & 1 deletion bundle/manifests/kuadrant-operator.clusterserviceversion.yaml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ metadata:
capabilities: Basic Install
categories: Integration & Delivery
containerImage: quay.io/kuadrant/kuadrant-operator:latest
createdAt: "2024-10-09T14:15:32Z"
createdAt: "2024-10-15T09:02:36Z"
description: A Kubernetes Operator to manage the lifecycle of the Kuadrant system
operators.operatorframework.io/builder: operator-sdk-v1.32.0
operators.operatorframework.io/project_layout: go.kubebuilder.io/v3
Expand Down
18 changes: 0 additions & 18 deletions bundle/manifests/kuadrant.io_authpolicies.yaml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -478,7 +478,6 @@ spec:
description: |-
Authentication configs.
At least one config MUST evaluate to a valid identity object for the auth request to be successful.
maxProperties: 10
type: object
authorization:
additionalProperties:
Expand Down Expand Up @@ -1066,7 +1065,6 @@ spec:
description: |-
Authorization policies.
All policies MUST evaluate to "allowed = true" for the auth request be successful.
maxProperties: 10
type: object
callbacks:
additionalProperties:
Expand Down Expand Up @@ -1354,7 +1352,6 @@ spec:
description: |-
Callback functions.
Authorino sends callbacks at the end of the auth pipeline to the endpoints specified in this config.
maxProperties: 10
type: object
metadata:
additionalProperties:
Expand Down Expand Up @@ -1684,7 +1681,6 @@ spec:
description: |-
Metadata sources.
Authorino fetches auth metadata as JSON from sources specified in this config.
maxProperties: 10
type: object
response:
description: |-
Expand Down Expand Up @@ -1896,7 +1892,6 @@ spec:
Custom success response items wrapped as HTTP headers.
For integration of Authorino via proxy, the proxy must use these settings to propagate dynamic metadata.
See https://www.envoyproxy.io/docs/envoy/latest/configuration/advanced/well_known_dynamic_metadata
maxProperties: 10
type: object
headers:
additionalProperties:
Expand Down Expand Up @@ -2097,7 +2092,6 @@ spec:
description: |-
Custom success response items wrapped as HTTP headers.
For integration of Authorino via proxy, the proxy must use these settings to inject data in the request.
maxProperties: 10
type: object
type: object
unauthenticated:
Expand Down Expand Up @@ -2678,7 +2672,6 @@ spec:
description: |-
Authentication configs.
At least one config MUST evaluate to a valid identity object for the auth request to be successful.
maxProperties: 10
type: object
authorization:
additionalProperties:
Expand Down Expand Up @@ -3266,7 +3259,6 @@ spec:
description: |-
Authorization policies.
All policies MUST evaluate to "allowed = true" for the auth request be successful.
maxProperties: 10
type: object
callbacks:
additionalProperties:
Expand Down Expand Up @@ -3554,7 +3546,6 @@ spec:
description: |-
Callback functions.
Authorino sends callbacks at the end of the auth pipeline to the endpoints specified in this config.
maxProperties: 10
type: object
metadata:
additionalProperties:
Expand Down Expand Up @@ -3884,7 +3875,6 @@ spec:
description: |-
Metadata sources.
Authorino fetches auth metadata as JSON from sources specified in this config.
maxProperties: 10
type: object
response:
description: |-
Expand Down Expand Up @@ -4096,7 +4086,6 @@ spec:
Custom success response items wrapped as HTTP headers.
For integration of Authorino via proxy, the proxy must use these settings to propagate dynamic metadata.
See https://www.envoyproxy.io/docs/envoy/latest/configuration/advanced/well_known_dynamic_metadata
maxProperties: 10
type: object
headers:
additionalProperties:
Expand Down Expand Up @@ -4297,7 +4286,6 @@ spec:
description: |-
Custom success response items wrapped as HTTP headers.
For integration of Authorino via proxy, the proxy must use these settings to inject data in the request.
maxProperties: 10
type: object
type: object
unauthenticated:
Expand Down Expand Up @@ -4869,7 +4857,6 @@ spec:
description: |-
Authentication configs.
At least one config MUST evaluate to a valid identity object for the auth request to be successful.
maxProperties: 10
type: object
authorization:
additionalProperties:
Expand Down Expand Up @@ -5453,7 +5440,6 @@ spec:
description: |-
Authorization policies.
All policies MUST evaluate to "allowed = true" for the auth request be successful.
maxProperties: 10
type: object
callbacks:
additionalProperties:
Expand Down Expand Up @@ -5738,7 +5724,6 @@ spec:
description: |-
Callback functions.
Authorino sends callbacks at the end of the auth pipeline to the endpoints specified in this config.
maxProperties: 10
type: object
metadata:
additionalProperties:
Expand Down Expand Up @@ -6064,7 +6049,6 @@ spec:
description: |-
Metadata sources.
Authorino fetches auth metadata as JSON from sources specified in this config.
maxProperties: 10
type: object
response:
description: |-
Expand Down Expand Up @@ -6275,7 +6259,6 @@ spec:
Custom success response items wrapped as HTTP headers.
For integration of Authorino via proxy, the proxy must use these settings to propagate dynamic metadata.
See https://www.envoyproxy.io/docs/envoy/latest/configuration/advanced/well_known_dynamic_metadata
maxProperties: 10
type: object
headers:
additionalProperties:
Expand Down Expand Up @@ -6475,7 +6458,6 @@ spec:
description: |-
Custom success response items wrapped as HTTP headers.
For integration of Authorino via proxy, the proxy must use these settings to inject data in the request.
maxProperties: 10
type: object
type: object
unauthenticated:
Expand Down
Loading

0 comments on commit 044e52b

Please sign in to comment.