Skip to main content

The ALTO Transport Information Publication Service
draft-ietf-alto-new-transport-22

Note: This ballot was opened for revision 16 and is now closed.

Erik Kline
No Objection
Comment (2023-10-24 for -17) Not sent
# Internet AD comments for draft-ietf-alto-new-transport-17
CC @ekline

* comment syntax:
  - https://github.com/mnot/ietf-comments/blob/main/format.md

* "Handling Ballot Positions":
  - https://ietf.org/about/groups/iesg/statements/handling-ballot-positions/

## Nits

### S1

* "there is no capability it transmits" ->
  "there is no capability for the server to transmit", perhaps?
Francesca Palombini
(was Discuss) No Objection
Comment (2023-12-11 for -21) Sent
Thank you for the work on this document.

Many thanks to Spencer Dawkins for his ART ART reviews (most recent being https://mailarchive.ietf.org/arch/msg/art/LibZiksz5-nO-g8IyOJrrtczj94/), and to Martin Thomson for his HTTPDir reviews (most recent being https://mailarchive.ietf.org/arch/msg/last-call/vz87ZLJVlbuVnSacxli8hvl-LTU/). Spencer and Martin's expertise has helped improve the document considerably, so thanks to them, and to the authors for considering their reviews.

Thank you for addressing my previous DISCUSS.
Jim Guichard
No Objection
John Scudder
No Objection
Comment (2023-10-25 for -17) Not sent
I support Francesca and Roman’s DISCUSS positions.
Murray Kucherawy
No Objection
Comment (2023-10-26 for -17) Sent
Thanks to Spencer Dawkins for his two ARTART reviews.

The answer to question #11 in the shepherd writeup is incomplete.

I support Francesca's DISCUSS position.  To answer her question about media types, I must admit I'm at a loss to understand this object notation and how it relates to a JSON payload.  A reference to decoding it would be helpful.  My best guess is that this one media type will be used for both cases, the payload will be something in JSON format in either case, and a consumer infers the semantics of the payload based on inspection of its structure, presumably returning an error if something unexpected is found.  However, this seems peculiar in the context of media types; I would prefer to see such division done using a parameter to the media type rather than inspection, since the whole idea of labeling a payload with a media type is to know what it is you're holding before you start doing something with it.

The SHOULDs in 6.2 are bare.  When might one legitimately deviate from that advice?  If no such reason exists, why aren't they MUSTs?  Same question about the SHOULDs in 4.1 and 8.5.

Regarding Sections 10.1 and 10.2, I suggest reviewing RFC 8126 for what's expected in "Interoperability Considerations".
Paul Wouters
No Objection
Comment (2023-10-24 for -17) Not sent
I support Roman's DISCUSS.
Roman Danyliw
(was Discuss) No Objection
Comment (2024-01-02 for -21) Sent
Thank you to Donald Eastlake for the SECDIR review.

Thanks for addressing my DISCUSS feedback.

** Section 6.2.  Editorial. New text was added clarifying text to prescribe that TIPS view URI must not be reused.  I would recommend being a clearer on this language.

OLD
      A server MUST NOT use a URI for different TIPS views, either for
      different resources or different request bodies to the same
      resource. 
NEW
A server MUST NOT use the same URI for different TIPS views, either for different resources or different request bodies to the same      resource. 


** Section 9.  
   The security considerations (Section 15 of [RFC7285]) of the base
   protocol fully apply to this extension.  For example, the same
   authenticity and integrity considerations (Section 15.1 of [RFC7285])
   still fully apply;

Since ALTO TIPS is a new protocol mechanism is it possible to improve on the TLS guidance in Section 8.3.5 of RFC7295 (from circa 2014)?  Specifically, can RFC9325 be mandated?
Zaheduzzaman Sarker
(was Discuss) No Objection
Comment (2023-12-01 for -20) Sent
The -20 version addresses my discuss comments. Thanks for resolving those. No objection from my side on this -20 version.
Éric Vyncke
Abstain
Comment (2023-10-25 for -17) Sent
# Éric Vyncke, INT AD, comments for draft-ietf-alto-new-transport-17

Thank you for the work put into this document even if I am not too satisfied by the whole document, hence my ABSTAIN position per "I cannot support sending this document forward." (see https://www.ietf.org/standards/process/iesg-ballots/). FYI, I stopped reviewing it after section 2 as my ballot position was already forged.

Please find below one blocking DISCUSS points (easy to address), some non-blocking COMMENT points (but replies would be appreciated even if only for my own education), and some nits.

Special thanks to Mohamed Boucadair for the shepherd's detailed write-up including the WG consensus and the justification of the intended status. 

Other thanks to Bob Halley, the Internet directorate reviewer (at my request), please consider this int-dir review:
https://datatracker.ietf.org/doc/review-ietf-alto-new-transport-16-intdir-telechat-halley-2023-10-19/ (review dated 19th of October and I have yet to read a reaction from the authors, comforting my ABSTAIN as I share Bob's concerns)

Other thanks to Wesley Eddy, the IoT directorate reviewer (at my request), please consider this iot-dir review:
https://datatracker.ietf.org/doc/review-ietf-alto-new-transport-17-iotdir-telechat-eddy-2023-10-23/ (and I have read Kai's reply)

I hope that this review helps to improve the document,

Regards,

-éric


# COMMENTS

## Abstract

The use of HTTP/1.1, HTTP/1.x, HTTP/2, HTTP/3 in the same paragraph is *really* confusing. Consider aligning the version to be consistent.

## Section 1

Like already written by other ballots, let's simply s/two transport protocols/two protocols/

In `and the server sends the complete content of the requested information (if any) resource to the client`, is it about an ALTO client or an application client ? 

## Section 2.1

`Incremental updates can reduce both the data storage on an ALTO server` hummm... in order to send increments, the server must keep all those increments for all its clients. I.e., increasing the data storage. Please either correct me (I would be happy) or the text.

Is 'prefetching' the right term ? It relates to client-initiated fetch rather than server-initiated push.

I strongly object (but not a block DISCUSS level) to `as many development tools and current ALTO implementations are based on HTTP/1.1.`. An IETF protocol design cannot be bound by 2023 implementations, e.g., else IPv6 would have never been standardized.

Where is the data model defined in `The key idea is to introduce a unified data model to describe the changes ` ?

# NITS

## Abstract

Should s/defines/define/ in `ALTO incremental updates using Server-Sent Events (SSE) (RFC 8895) defines a multiplexing protocol on top of HTTP/1.x`

## TIPS

I am not a native English speaker, and it is probably too late, but s/Transport Information Publication Service/Information Transport Publication Service/ sounds better to my French-speaking ears.
Martin Duke Former IESG member
Yes
Yes (for -16) Unknown

                            
Andrew Alston Former IESG member
No Objection
No Objection (for -17) Not sent

                            
Robert Wilton Former IESG member
No Objection
No Objection (2023-10-26 for -17) Sent
Hi,

Thank you for this document.  I haven't reviewed this document that closely, and it might be interesting to know whether there are interoperable implementations of this specification, or where there may still be issues lurking, or more precise text to describe its behavior and APIs.  Hence, I was wondering whether it would be better from this document to be published as an experimental RFC rather than Proposed Standard, this would then leave a lot more flexibility for making more substantial changes to the protocol specification in future, if needed.

Regards,
Rob