Skip to content

Implement frame-ancestors directive - #69

Merged
notriddle merged 1 commit into
rust-ammonia:masterfrom
TimvdLippe:frame-ancestors
Mar 25, 2026
Merged

Implement frame-ancestors directive#69
notriddle merged 1 commit into
rust-ammonia:masterfrom
TimvdLippe:frame-ancestors

Conversation

@TimvdLippe

Copy link
Copy Markdown
Contributor

This requires implementing the navigation response check, which is only partially implemented. Even though the spec has a step 3 to check the navigation request, there is no directive that checks for it.

Also, the logic for traversing navigable isn't possible in this crate. Instead, the caller is expected to pass in the list of origins as URLs, so that this crate can loop over them.

This requires implementing the navigation response check, which
is only partially implemented. Even though the spec has a step 3
to check the navigation request, there is no directive that
checks for it.

Also, the logic for traversing navigable isn't possible in this
crate. Instead, the caller is expected to pass in the list of
origins as URLs, so that this crate can loop over them.
@notriddle
notriddle merged commit 2fc4662 into rust-ammonia:master Mar 25, 2026
8 checks passed
@TimvdLippe

Copy link
Copy Markdown
Contributor Author

Thanks once again! Can you publish 0.7.1 so that I can update our version in Servo?

@TimvdLippe

Copy link
Copy Markdown
Contributor Author

Ah and in that case, #70 can also be included. Coincidentally found that while working on a different issue.

@TimvdLippe

Copy link
Copy Markdown
Contributor Author

@notriddle sorry to keep on bothering you. Unfortunately for Servo we need a new release of the crate before we can use it. Typically we update the crate with some fixes and improved WPT tests. Then we find new fixes that potentially require more updates to the crate.

Previously we used a git fork for rapid iteration and then to upstream. However, the updates had stopped at the time and we no longer want git dependencies. That's because there are plans to publish Servo itself as a crate.

Therefore, for now we require crate publications after fixes so that we can pull them in. Can you publish 0.7.1 and let us know what your thoughts are on the process?

@TimvdLippe
TimvdLippe deleted the frame-ancestors branch March 26, 2026 06:39
@notriddle

Copy link
Copy Markdown
Member

Okay, it's pushed.

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