Skip to content

rfc: user data domain separation - #12

Merged
alanshaw merged 1 commit into
mainfrom
ash/rfc/user-data-domain-separation
Jul 23, 2026
Merged

rfc: user data domain separation#12
alanshaw merged 1 commit into
mainfrom
ash/rfc/user-data-domain-separation

Conversation

@alanshaw

@alanshaw alanshaw commented Jul 14, 2026

Copy link
Copy Markdown
Member

📖 Preview

Summary

Move <region>.s3.fil.one to <region>.s3.filone.com.

## Open Questions

- Is `filone.com` the right domain?
- PSL scope: list `filone.com` itself or `s3.filone.com`?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What do you think about using <bucket_name>.<region>.s3.filone.com/<bucket_name>? That way we could:

  • Isolate the malicious bucket flagged by PSL without flagging the top domain
  • Keep the bucket name in the path

This would probably require an extra check performed by the gateway to make sure that path and domain bucket name match before serving the request.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think that makes it harder to switch to pure bucket-in-subdomain when we want to do public buckets...

Do you know what the reason for using path based was originally? I think we should switch to subdomain based ASAP.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't know the exact reason to be honest, but I believe it was related to service providers not supporting the feature. Would be good to double check with them.

@pyropy pyropy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This proposal sounds reasonable to me. However I'd like to discuss usage of bucket names in the subdomain which could probably grow into it's own RFC.

@bajtos bajtos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👏🏻

I agree we should support including the bucket name in the domain name (instead of the path) ASAP, but I think that's a small, incremental addition to this proposal that's not blocking this PR from landing.

@alanshaw
alanshaw merged commit 148ed2b into main Jul 23, 2026
3 checks passed
@alanshaw
alanshaw deleted the ash/rfc/user-data-domain-separation branch July 23, 2026 12:12
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.

4 participants