Skip to content

Commit

Permalink
ingress-controller/settings: remove cookie secure option (#872)
Browse files Browse the repository at this point in the history
* ingress-controller/settings: remove cookie secure option

* fix newlines
  • Loading branch information
calebdoxsey authored Jan 12, 2024
1 parent b108dba commit 619f07c
Show file tree
Hide file tree
Showing 6 changed files with 1 addition and 35 deletions.
4 changes: 0 additions & 4 deletions apis/ingress/v1/pomerium_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -163,10 +163,6 @@ type Cookie struct {
// If you specify the domain explicitly, then subdomains would also be included.
// +optional
Domain *string `json:"domain,omitempty"`
// Secure if set to false, would make a cookie accessible over insecure protocols (HTTP).
// Defaults to <code>true</code>.
// +optional
Secure *bool `json:"secure,omitempty"`
// HTTPOnly if set to <code>false</code>, the cookie would be accessible from within the JavaScript.
// Defaults to <code>true</code>.
// +optional
Expand Down
5 changes: 0 additions & 5 deletions apis/ingress/v1/zz_generated.deepcopy.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 0 additions & 4 deletions config/crd/bases/ingress.pomerium.io_pomerium.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -122,10 +122,6 @@ spec:
- lax
- none
type: string
secure:
description: Secure if set to false, would make a cookie accessible
over insecure protocols (HTTP). Defaults to <code>true</code>.
type: boolean
type: object
identityProvider:
description: IdentityProvider configure single-sign-on authentication
Expand Down
4 changes: 0 additions & 4 deletions deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -130,10 +130,6 @@ spec:
- lax
- none
type: string
secure:
description: Secure if set to false, would make a cookie accessible
over insecure protocols (HTTP). Defaults to <code>true</code>.
type: boolean
type: object
identityProvider:
description: IdentityProvider configure single-sign-on authentication
Expand Down
1 change: 0 additions & 1 deletion pomerium/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,6 @@ func applyCookie(_ context.Context, p *pb.Config, c *model.Config) error {
p.Settings.CookieDomain = c.Spec.Cookie.Domain
p.Settings.CookieName = c.Spec.Cookie.Name
p.Settings.CookieHttpOnly = c.Spec.Cookie.HTTPOnly
p.Settings.CookieSecure = c.Spec.Cookie.Secure
if c.Spec.Cookie.Expire != nil {
p.Settings.CookieExpire = durationpb.New(c.Spec.Cookie.Expire.Duration)
}
Expand Down
18 changes: 1 addition & 17 deletions reference.md
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ PomeriumSpec defines Pomerium-specific configuration parameters.
</p>
<p>

PassIdentityHeaders sets the <a href="https://www.pomerium.com/docs/reference/routes/pass-identity-headers">pass identity headers</a> option.
PassIdentityHeaders sets the <a href="https://www.pomerium.com/docs/reference/pass-identity-headers">pass identity headers</a> option.
</p>

</td>
Expand Down Expand Up @@ -423,22 +423,6 @@ Cookie defines Pomerium session cookie options.
</td>
</tr>

<tr>
<td>
<p>
<code>secure</code>&#160;&#160;

<strong>boolean</strong>&#160;

</p>
<p>

Secure if set to false, would make a cookie accessible over insecure protocols (HTTP). Defaults to <code>true</code>.
</p>

</td>
</tr>

</tbody>
</table>

Expand Down

0 comments on commit 619f07c

Please sign in to comment.