Last Call Review of draft-ietf-appsawg-text-markdown-use-cases-02
review-ietf-appsawg-text-markdown-use-cases-02-secdir-lc-wouters-2015-07-02-00
| Request | Review of | draft-ietf-appsawg-text-markdown-use-cases |
|---|---|---|
| Requested revision | No specific revision (document currently at 07) | |
| Type | IETF Last Call Review | |
| Team | Security Area Directorate (secdir) | |
| Deadline | 2015-07-03 | |
| Requested | 2015-06-25 | |
| Authors | Sean Leonard | |
| I-D last updated | 2016-03-22 (Latest revision 2015-09-04) | |
| Completed reviews |
Genart IETF Last Call review of -02
by Tom Taylor
(diff)
Secdir IETF Last Call review of -02 by Paul Wouters (diff) Opsdir IETF Last Call review of -02 by Dan Romascanu (diff) |
|
| Assignment | Reviewer | Paul Wouters |
| State | Completed | |
| Request | IETF Last Call review on draft-ietf-appsawg-text-markdown-use-cases by Security Area Directorate Assigned | |
| Reviewed revision | 02 (document currently at 07) | |
| Result | Has issues | |
| Completed | 2015-07-02 |
review-ietf-appsawg-text-markdown-use-cases-02-secdir-lc-wouters-2015-07-02-00
I have reviewed this document as part of the security directorate's
effort to review all IETF documents being processed by the IESG.
These comments were written primarily for the benefit of the security
area directors. Document editors and WG chairs should treat these
comments just like any other last call comment.
This document describes use cases and sometimes existing deployed code
on handling "markdown" text. As such, the document introduces no new
security considerations, and the Security Considerations section points
to other documents that further document the respective markdown
variants and their own security considerations.
Recommendation: Ready with Issues
I wanted to point out two use cases (or existing deployed code?)
that uses some features that might be considered a security issue.
2.1 talks about filesystem "extended attributes" and suggests to add a
resource named "variant". This name might be a little too generic to
only apply to markdown and might cause a name spaec collision that
could potentially be a security risk. If this is a use case without
deployed code, I would recommend renaming this resource to something
more specific, eg "markdown-varient". If it describes actual code,
then I guess that ship has sailed.
2.4 talks about MIME aware clients saving a "batch script" to disk for
later execution. These kind of "autorun" or "preview" features are
a security nightmare, so here too I would hope this has not yet been
coded. And if not, to reconsider not supporting such a feature.
Paul