Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spaces in authentication string for yaz-client #101

Closed
dd217 opened this issue Nov 15, 2023 · 1 comment
Closed

Spaces in authentication string for yaz-client #101

dd217 opened this issue Nov 15, 2023 · 1 comment

Comments

@dd217
Copy link

dd217 commented Nov 15, 2023

Hello,
I'm working with a vendor that has a z39.50 target that requires a password. The password we were given by the vendor contains a space. My expectation was that the the code to parse the authentication string would respect single or double quotes so that a password could contain a space. Looking at nmem_strsplit_blank in nmemsdup.c, which is called for tokenizing the authentication string into the username and password, it looks like we're just splitting on any space regardless if part of a string was put in quotes.

Is this something that should actually be addressed, or is this outside the spec of z39.50/ZOOM.

adamdickmeiss added a commit that referenced this issue Jan 27, 2025
Document the yaz-client command auth.
adamdickmeiss added a commit that referenced this issue Jan 27, 2025
Allow spaces for auth strings in yaz-client #101

Document the yaz-client command auth.
@adamdickmeiss
Copy link
Contributor

We'll have this done in next release.. You can backslash your way out eg

auth user the-password-with\ two\ blanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants