diff options
Diffstat (limited to '')
| -rw-r--r-- | src/constant.rs | 11 | ||||
| -rw-r--r-- | src/construct/label_end.rs | 4 |
2 files changed, 10 insertions, 5 deletions
diff --git a/src/constant.rs b/src/constant.rs index 5cb7826..fc74316 100644 --- a/src/constant.rs +++ b/src/constant.rs @@ -193,10 +193,15 @@ pub const HTML_RAW_SIZE_MAX: usize = 8; /// To safeguard performance, labels are capped at a large number: `999`. pub const LINK_REFERENCE_SIZE_MAX: usize = 999; -/// To do. +/// Maximum allowed unbalanced parens in destination. +/// +/// There can be many balanced parens, but if there are 33 opens that were not +/// yet closed, the destination does not parse. +/// `CommonMark` requires that at least 3 opening parens are allowed. /// See: <https://spec.commonmark.org/0.30/#link-destination>, -/// <https://github.com/remarkjs/react-markdown/issues/658#issuecomment-984345577>. -pub const LINK_RESOURCE_DESTINATION_BALANCE_MAX: usize = 32; +/// In practice, this is quite low, and several places instead cap it at 32. +/// See: <https://github.com/remarkjs/react-markdown/issues/658#issuecomment-984345577>. +pub const RESOURCE_DESTINATION_BALANCE_MAX: usize = 32; /// List of protocols allowed, when operating safely, as `href` on `a`. /// diff --git a/src/construct/label_end.rs b/src/construct/label_end.rs index 6e8e476..a814302 100644 --- a/src/construct/label_end.rs +++ b/src/construct/label_end.rs @@ -1,6 +1,6 @@ //! To do -use crate::constant::LINK_RESOURCE_DESTINATION_BALANCE_MAX; +use crate::constant::RESOURCE_DESTINATION_BALANCE_MAX; use crate::construct::{ partial_destination::{start as destination, Options as DestinationOptions}, partial_label::{start as label, Options as LabelOptions}, @@ -436,7 +436,7 @@ fn resource_open(tokenizer: &mut Tokenizer, code: Code) -> StateFnResult { t, c, DestinationOptions { - limit: LINK_RESOURCE_DESTINATION_BALANCE_MAX, + limit: RESOURCE_DESTINATION_BALANCE_MAX, destination: TokenType::ResourceDestination, literal: TokenType::ResourceDestinationLiteral, marker: TokenType::ResourceDestinationLiteralMarker, |