ctubio / Krypto-trading-bot

Self-hosted crypto trading bot (automated high frequency market making) written in C++
https://127.0.0.1:3000
Other
3.36k stars 821 forks source link

Matryoshka: nested client won't connect if server has password with special characters #453

Closed gladstonedigital closed 6 years ago

gladstonedigital commented 6 years ago

In K2.sh:

UI_PASSWORD="somepassword!with)and("

I'm able to access the second instance in the browser at its own URL just fine, but when I try to add it as a nested matryoshka instance in the first instance, "localhost refused to connect"

However, when I change the password to use a normal password with only letters, it connects just fine.

ctubio commented 6 years ago

are you adding to the url the username and the password instead of just the ip letting the browser prompt the credentials?

your url is like https://user:pass@ip? or is https://ip? thanks for confirm''

gladstonedigital commented 6 years ago

No I didn't try that, I was just using url like https://localhost:3001, and letting the browser prompt for password. It might work the way you mentioned

ctubio commented 6 years ago

letting the browser prompt for the password is what i usualy do, and i could not reproduce the issue you describe; for me the passwords works nice as long as the browser prompts for it and i dont put it wrong :S

i will test a bit more laterz before close this

gladstonedigital commented 6 years ago

Hm that's strange, I tried it multiple times and it kept failing, but worked if I went to the second instance's page directly. I'll play with it more later

ctubio commented 6 years ago

im sorry im not able to reproduce this