update for #576#581
Conversation
| and so the <a href="https://www.w3.org/TR/mathml-core/#security-considerations">Security Considerations</a> specified there apply.</p> | ||
| and so the <a href="https://www.w3.org/TR/mathml-core/#security-considerations">Security Considerations</a> specified there apply in addition to the | ||
| considerations listed below.</p> | ||
| <p>In some situations, MathML expressions can be parsed as XML. The security considerations of XML parsing apply then as explained in [[?RFC7303]].</p> |
There was a problem hiding this comment.
Is [[?RFC7303]] correct or is the "?" someone guessing as to the reference? It references XML Media Types, which seems wrong.
There was a problem hiding this comment.
It's respec markup I forget exactly what the ? does but we use it quite a few times throughout the spec It affects the form of the generated bibliography. That part was in the existing document not added by this PR @NSoiffer
There was a problem hiding this comment.
Do you think the reference to media types is correct (in regards to XML parsing security considerations)?
|
I think we also were going to do a small update to the intent section. Should that be a separate PR? |
@NSoiffer er.. I forget, do you want to make a PR? It's probably easier than reminding me what I was supposed to add. |
2 participants
This implements in full (I hope) the suggestions from #576