-
Notifications
You must be signed in to change notification settings - Fork 696
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
Fail to login when password has special chars #15
Comments
I was going to make a new post, but my error is possibly the same as above. I use an email address for login and a password that ends with a special character (one of these: !@#$%^&*()). When I attempt to use the script, I get the following:
|
Ok, I'll work on it. |
Sorry, I should have give an example: if “mail@mail.com:pa$$word” is correctly parsed, then it seems good to me |
Ok, updated. |
No, you know. You have to escape the "$" char: "pa\$\$word"... |
That's what I thought, perhaps on note on the README could help future users ? Anyway, thanks for having a look at it |
Done with a6bbf31 |
Hi, My password includes several special characters, tried escaping all of them but still getting the error [#] API: Out-of-Session (Authentication credentials were not provided.). special characters I use: @%^$ Also some of them comes one after the other so double escaping needed %$ Due to some other reasons I cannot change my password easily.. Any advice? |
Fixed invalid filename
Hi
|
I am getting the same behavior. I have removed special characters from my password. |
maybe you can try the |
First, thank you for this, it's going to be extremely valuable to me (reading on an e-reader is so much more comfortable than on a tablet or a phone)
Now, my problem is my passwords are generated, and, as such, contain special chars.
Right now, two of them are giving me problems:
:
and$
As a workaround, I escaped
$
with\
, but I had to modify credentials parsing to change the configured separator into a character which does not appear in my password.May I suggest adding a
--sep
option, defaulting to:
, so that user can choose whatever char suits best ?As for
$
, I think it's more of a shell related problem, and I can't see for now a solution to this (other than escaping it, that is). Do you have any idea how we could overcome this ?I am not fluent in python, but I can probably submit a PR making this separator option change, what do you think ?
The text was updated successfully, but these errors were encountered: