Skip to content

chore: bump deps - #203

Merged
miparnisari merged 1 commit into
mainfrom
update-spicedb
Aug 21, 2026
Merged

chore: bump deps#203
miparnisari merged 1 commit into
mainfrom
update-spicedb

Conversation

@miparnisari

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added area/dependencies Affects dependencies area/core labels Aug 20, 2026
@github-actions github-actions Bot added the area/tooling Affects the dev or user toolchain label Aug 20, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@miparnisari
miparnisari marked this pull request as ready for review August 20, 2026 23:58

@tstirrat15 tstirrat15 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread e2e/go.mod Outdated
buf.build/gen/go/prometheus/prometheus/protocolbuffers/go v1.36.10-20251118093737-4105057cc7d4.1 // indirect
buf.build/go/protovalidate v1.1.3 // indirect
cel.dev/expr v0.25.1 // indirect
buf.build/go/protovalidate v1.2.0 // indirect

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1.3? Or are we not worried about that?

Comment thread pkg/proxy/server.go
req.URL.Scheme = "https"
req.Out.URL.Host = strings.TrimSuffix(host, "/")
req.Out.URL.Scheme = "https"
req.SetXForwarded()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this change do?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see https://pkg.go.dev/net/http/httputil#ReverseProxy.Director

Director used to preserve X-Forwarded-For. Rewrite does not so we manually add it

@tstirrat15 tstirrat15 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTNM

@miparnisari
miparnisari merged commit db5183a into main Aug 21, 2026
7 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area/core area/dependencies Affects dependencies area/tooling Affects the dev or user toolchain

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants