Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Early wide review of IFT #33

Open
7 of 8 tasks
svgeesus opened this issue Sep 14, 2021 · 24 comments
Open
7 of 8 tasks

Early wide review of IFT #33

svgeesus opened this issue Sep 14, 2021 · 24 comments
Assignees

Comments

@svgeesus
Copy link
Contributor

svgeesus commented Sep 14, 2021

This meta-issue tracks the early (post-FPWD) wide review of IFT

@svgeesus svgeesus self-assigned this Oct 8, 2021
@svgeesus
Copy link
Contributor Author

svgeesus commented Feb 15, 2022

@svgeesus
Copy link
Contributor Author

Security should be given a second change to review, once we publish the merged spec (they never got a chance to look at the FPWD of the separate Range Request spec)

@svgeesus
Copy link
Contributor Author

The two open privacy issues need further discussion, because I think we are talking at cross purposes, but I am optimistic that they can be closed relatively quickly

@svgeesus
Copy link
Contributor Author

svgeesus commented May 17, 2022

I'd like to request TAG review once we publish the merged spec, though I think it would be preferable to do the network perf review first, to save them telling us to go do that :)

@svgeesus
Copy link
Contributor Author

Adding network performance review as suggested in #74 by @mnot

@svgeesus
Copy link
Contributor Author

svgeesus commented Jun 2, 2022

Further review progress pending on /TR publication of merged spec

@svgeesus
Copy link
Contributor Author

@svgeesus
Copy link
Contributor Author

I18n review is closed

@svgeesus
Copy link
Contributor Author

Web Perf WG review requested 28 June 2022

@svgeesus
Copy link
Contributor Author

IETF HTTP WG review requested 28 June 2022

@svgeesus
Copy link
Contributor Author

svgeesus commented Jul 1, 2022

On July 7 Perf WG Agenda

@svgeesus
Copy link
Contributor Author

svgeesus commented Jul 4, 2022

Response from Martin Thomson raises several issues, which we should track as individual GH issues.

Apparently the rationale from the PFE report was not obvious, and I responded on that already and will add an issue.

@svgeesus
Copy link
Contributor Author

svgeesus commented Jul 9, 2022

@svgeesus
Copy link
Contributor Author

@svgeesus
Copy link
Contributor Author

@svgeesus
Copy link
Contributor Author

svgeesus commented Mar 8, 2023

There are zero open privacy issues so marking privacy review as completed.

@svgeesus
Copy link
Contributor Author

svgeesus commented Mar 9, 2023

All the issues from Martin Thomson are now closed, so marking the IETF HTTP WG Review as closed

@svgeesus
Copy link
Contributor Author

svgeesus commented Mar 9, 2023

@vlevantovsky @garretrieger my feeling is that we should wrap up any outstanding edits and then republish the spec before asking for TAG review. I also need to write an explainer before we ask for their review.

Security has had plenty of time to respond, we don't need to wait longer; marking their review as complete due to timeout.

@garretrieger
Copy link
Contributor

Sounds good, we can discuss what's left to do at the next working group call. Should we also split out the range request specification before republishing and asking for a TAG review?

@svgeesus
Copy link
Contributor Author

Should we also split out the range request specification before republishing and asking for a TAG review?

I think so, there are a bunch of issues tagged Range Request and it is advancing more slowly than Patch Subset.

@garretrieger
Copy link
Contributor

OK I took a pass at splitting here: #138

I kept the split simple, just moved the range request specific section out while keeping the range request bits in the "common" section around in the main document.

@svgeesus
Copy link
Contributor Author

I think that was a good call.

@svgeesus
Copy link
Contributor Author

TAG review requested 26 May 2023

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

No branches or pull requests

2 participants