Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@wadahiro
Copy link
Contributor

@wadahiro wadahiro commented Jun 8, 2025

Change summary

The parser now correctly handles:

  • Quoted values containing spaces (e.g., "dc ba")
  • Quoted values with escaped quotes (e.g., "say \"hello\" world")

Previously these patterns would cause parsing errors.

Checklist

  • This PR contains no AI generated code
  • book chapter included (if relevant)
  • design document included (if relevant)

The parser now correctly handles:
- Quoted values containing spaces (e.g., "dc ba")
- Quoted values with escaped quotes (e.g., "say \"hello\" world")

Previously these patterns would cause parsing errors.
Copy link
Member

@Firstyear Firstyear left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@github-project-automation github-project-automation bot moved this from 🆕 New to 🔖 Ready in Organising Everything Jun 11, 2025
@Firstyear Firstyear enabled auto-merge (squash) June 11, 2025 00:02
@Firstyear Firstyear merged commit 3de9bf5 into kanidm:master Jun 11, 2025
20 of 22 checks passed
@github-project-automation github-project-automation bot moved this from 🔖 Ready to ✅ Done in Organising Everything Jun 11, 2025
Firstyear pushed a commit that referenced this pull request Jun 11, 2025
…es (#3673)

The parser now correctly handles:
- Quoted values containing spaces (e.g., "dc ba")
- Quoted values with escaped quotes (e.g., "say \"hello\" world")

Previously these patterns would cause parsing errors.
@wadahiro wadahiro deleted the fix-filter branch June 11, 2025 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants