Skip to main content

Last Call Review of draft-ietf-httpbis-rfc7238bis-02
review-ietf-httpbis-rfc7238bis-02-genart-lc-krishnan-2015-02-04-00

Request Review of draft-ietf-httpbis-rfc7238bis
Requested revision No specific revision (document currently at 03)
Type IETF Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2015-01-28
Requested 2015-01-15
Authors Julian Reschke
I-D last updated 2015-10-14 (Latest revision 2015-02-05)
Completed reviews Genart IETF Last Call review of -02 by Suresh Krishnan (diff)
Secdir IETF Last Call review of -02 by Sandra L. Murphy (diff)
Opsdir IETF Last Call review of -02 by Sarah Banks (diff)
Assignment Reviewer Suresh Krishnan
State Completed
Request IETF Last Call review on draft-ietf-httpbis-rfc7238bis by General Area Review Team (Gen-ART) Assigned
Reviewed revision 02 (document currently at 03)
Result Ready
Completed 2015-02-04
review-ietf-httpbis-rfc7238bis-02-genart-lc-krishnan-2015-02-04-00
I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at
<

http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>



Please wait for direction from your document shepherd or AD before 


posting a new version of the draft.




Document: draft-ietf-httpbis-rfc7238bis-02.txt
Reviewer: Suresh Krishnan
Review Date: 2015/02/03
IESG Telechat date: 2015/02/05



Summary: This draft is ready for publication as a Proposed Standard. I 


do have a nit you may want to look at, but feel free to ignore.




Nit:



As it is just a republication of the experimental RFC7238, I went over 


and ran the tests at






http://greenbytes.de/tech/tc/httpredirects/#l-308





with both Firefox and Chrome on Linux. When I ran the t308methods test 


which stated "method should be preserved in all cases", I got an error 


for the TRACE method. This is probably due to the fact that the test 


server has disabled TRACE. I think it would be nice to remove TRACE from 


the test suite so that an ugly error would not show up :-). Or 


alternatively, if support for TRACE redirects is required, enable it on 


the server so that it can be tested




Thanks
Suresh