You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Misleading error message when appending queries #414
I am using trurl to append fields to a query to avoid url encoding them.
From the reading the man page, not paying enough attention and misinterpreting Long options can be provided either as --flag argument or as --flag=argument and -a, --append [component]=[data], I ran
While the examples in the docs do use query=key=value, it threw me when it said query wasn't supported. Could the error be something like component query missing data or similar?
Thanks, and thanks for trurl in general, really helpful tool.
Perhaps there should be a hierarchy for some of the errors, where we ensure the flag is formatted properly, something like --append expected '=' after 'query' and then if it knows its in the right form we can make judgements on whether it's supported or not.
Hi, I’d like to take this on. I’ve reproduced the misleading message and prepared a small fix that reports a missing = before checking the append component, with regression tests. The available test suite and sanitizer checks pass locally. Please let me know if someone is already working on it.
Hi maintainers, the proposed diagnostic fix and regression tests are in #454. I'd appreciate a review when you have a moment, and I'm happy to make any changes needed. Thank you!
Activity
jacobmealey commentedon Nov 25, 2025
Perhaps there should be a hierarchy for some of the errors, where we ensure the flag is formatted properly, something like
--append expected '=' after 'query'and then if it knows its in the right form we can make judgements on whether it's supported or not.rajmodi8905 commentedon Oct 9, 2026
Hi, I’d like to take this on. I’ve reproduced the misleading message and prepared a small fix that reports a missing = before checking the append component, with regression tests. The available test suite and sanitizer checks pass locally. Please let me know if someone is already working on it.
rajmodi8905 commentedon Oct 27, 2026
Hi maintainers, the proposed diagnostic fix and regression tests are in #454. I'd appreciate a review when you have a moment, and I'm happy to make any changes needed. Thank you!