Skip to content

Conversation

@chait-slim
Copy link
Contributor

Should be merged after: #4397

Signed-off-by: Chai Tadmor <chai.tadmor@root.io>
@jess-lowe jess-lowe changed the title Add source test feat(source): Add Root source test Jan 4, 2026
@jess-lowe
Copy link
Contributor

/gcbrun

Copy link
Contributor

@jess-lowe jess-lowe left a comment

Choose a reason for hiding this comment

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

LGTM!

@jess-lowe jess-lowe merged commit 91c3ca1 into google:master Jan 4, 2026
19 checks passed
@jess-lowe
Copy link
Contributor

Hey @chait-slim, we attempted to start ingesting into the test instance yesterday, but are getting 405 errors. Would it be possible to add 'HEAD' to the allowed actions list? We do HEAD requests before GET requests to check whether anything actually needs updating to reduce traffic on your end :) Thanks!

@chait-slim
Copy link
Contributor Author

Hi @jess-lowe I have added a HEAD action:

HTTP/2 200
date: Tue, 06 Jan 2026 14:21:42 GMT
content-type: application/octet-stream
content-length: 792644
etag: "b8693ab49d2d3cc75f53a7c97dd2df81"
last-modified: Tue, 06 Jan 2026 12:00:09 GMT
vary: Origin
cf-cache-status: DYNAMIC
report-to: {"group":"cf-nel","max_age":604800,"endpoints":[{"url":"https://a.nel.cloudflare.com/report/v4?s=z0Y671M4LwGUJcc%2FB0W8j7IKmMHeGZADD43Sd9%2FnM%2B2XkuWzQaKEWNlBT3FTT%2BFHtJdt8ho1DNX5yCVJYJx5PCTEu2AtanlUROY%3D"}]}
nel: {"report_to":"cf-nel","success_fraction":0.0,"max_age":604800}
server: cloudflare
cf-ray: 9b9bdd01c818b2ed-TLV
alt-svc: h3=":443"; ma=86400

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