Skip to content

Clarify informative notes about cross-origin#424

Open
noamr wants to merge 4 commits intogh-pagesfrom
tao
Open

Clarify informative notes about cross-origin#424
noamr wants to merge 4 commits intogh-pagesfrom
tao

Conversation

@noamr
Copy link
Contributor

@noamr noamr commented Feb 10, 2026

Closes #423.


Preview | Diff

index.html Outdated
to the empty string.
{{PerformanceResourceTiming/secureConnectionStart}}. Further, the
{{PerformanceResourceTiming/nextHopProtocol}} attribute will be set
to the empty string.
Copy link
Member

Choose a reason for hiding this comment

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

I think the wording is still somewhat confusing here as the attributes are not set. Rather the getters end up returning 0 or the empty string or whatever due to the underlying data not revealing anything.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Reworded a bit

data-cite="HTML#cors-cross-origin">CORS-cross-origin</a>.

{{PerformanceResourceTiming/transferSize}} is affected both by the <a data-cite="FETCH#concept-tao-check">timing
allow check</a> and by the <a data-cite="HTML#cors-cross-origin">CORS-cross-origin</a> status.
Copy link
Member

Choose a reason for hiding this comment

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

It's mostly a function of encodedBodySize so wouldn't this apply to encodedBodySize as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't quite understand?

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

transferSize is a function of something that comes from response's body info and cache state. cache state is specifically zeroed by TAO-fail here (4.2.5) and body-info isn't.

Copy link
Member

Choose a reason for hiding this comment

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

Right, but that means the timing allow check equally applies to encodedBodySize, no?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, encodedBodySize is only affected by CORS check as it doesn't look at cache state.

Copy link
Member

Choose a reason for hiding this comment

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

Oh I see, the cache state is zeroed due to allow timing.

Where is body info masked though?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

of fetching the resource, and do not go through the <a data-cite="FETCH#concept-tao-check">timing
allow check</a>.
To get the full information about those fetches, the service worker's own performance timeline
can be inspected. [[SERVICE-WORKERS]]
Copy link
Member

Choose a reason for hiding this comment

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

Some hint of where the requirements can be found about how the forwarding happens would be useful I think.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

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.

PerformanceResourceTiming transferSize getter steps does not consider cross origin resources.

2 participants