lxbme commented on PR #13078:
URL: https://github.com/apache/apisix/pull/13078#issuecomment-4257850305
Hi, @nic-6443. I believe the existing regression issues require a more
long-term discussion. Currently, the following issues have been identified:
1. Handling of `"` and `\n` in double-quoted (`dquote`) scalars, as well as
support for multi-line double-quoted strings.
2. Handling of the `'` character within single-quoted (`squote`) scalars.
3. Handling of `#` or flow indicators (e.g., `,`, `[]`, `{}`) within plain
scalars.
Additionally, several scenarios need clarification:
- How block scalars (`|`, `>`) should be supported.
- Whether variable replacement should occur within comment contexts (e.g.,
`# ${{V}}`).
I am concerned about whether the current pre-parsing approach is truly
appropriate. It introduces numerous potential regressions, and addressing them
might necessitate substantial changes to the existing logic. I wonder if this
exceeds the scope of the original issue? I believe further discussion and
evaluation are necessary.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
