Document: draft-ietf-scim-cursor-pagination Title: Cursor-based Pagination of SCIM Resources Reviewer: Bo Wu Review result: Has Nits Hi all, Thanks the authors for the document. I'm the assigned OPSDIR reviewer of this document. I think there are some editorial nits can be improved. Summary: This document describes cursor-based pagination extensions to the SCIM (System for Cross-Domain Identity Management) protocol. Major Issues: None Minor Issues: None Nits: 1.Section 2.1: Cursor Expiration Consistency (Section 2.1 vs. Section 4) • Issue: Table 3 uses a fixed 600-second example for expiredCursor while Section 4 defines configurable cursorTimeout. • Recommendation: Revise the Table 3 description to: "Do not wait longer than the service provider's configured cursorTimeout value to request additional pages." 2.Section 2.1: InvalidCount Error Consistency as the previous one • Recommendation: Revise the Table 3 description to: "Count value must be between 1 and the service provider's configured maxPageSize." 3.Section 4: Missing Pagination Attribute Definition The document appears to lack a formal definition for the pagination complex attribute. • Recommendation: Add JSON schema definitions for the pagination attribute structure. 4.Section 5: Security Considerations: No explicit reference to RFC 7644 Section 7 security framework. • Recommendation: Add introductory text in Section 5: "This specification inherits all security considerations from RFC 7644 Section 7, with the following additions..." 5.Section 5: Terminology Consistency • Issue: Inconsistent use of "client" (RFC 7644) vs. "actor" (Section 5.1). • Recommendation: Replace all "actor" references with "client" in security considerations. 6. Replace "This draft" with "This document" throughout the text. Thanks, Bo Wu -- last-call mailing list -- last-call@xxxxxxxx To unsubscribe send an email to last-call-leave@xxxxxxxx
