-
Notifications
You must be signed in to change notification settings - Fork 42
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
"Invalid signature" appears at random from different commands #26
Comments
Hi Beau, Can you dump your node version and the version of this library you're using? |
|
Ok, I'm getting an ubuntu machine up to see if I can repro; anecdotally I couldn't on smartos or mac. |
Oh, and I should have asked - are you using SSH keys locally or with the agent, and is your key DSA or RSA (and either way, what size)? |
Here's SDC_CLI_ACCOUNT=singlyinc I should note that this morning it took ~50 iterations of that repro before I saw the issue. If you have logging on the Joyent end the IP I'm coming from is 173.255.212.44. On Wednesday, March 27, 2013 at 12:59 PM, Mark Cavage wrote:
|
Ok - I was able to get this to happen from an ubuntu client - I'm looking On Wed, Mar 27, 2013 at 1:31 PM, Beau Gunderson [email protected]:
|
Hi Beau, Just to confirm, I can repro this, but I need some debug logs from production to diagnose further - which currently requires a restart, so this will take a little bit to get to the bottom of - I'm sorry for the trouble this is causing you; this is definitely not expected. m |
Glad you were able to confirm, thanks for looking into this! :) I have some code in place to retry if an sdc- command fails with Invalid signature so I'm OK for the short-term. On Wednesday, March 27, 2013 at 3:34 PM, Mark Cavage wrote:
|
Hi Beau, Just as an update, since it's been a couple days, I confirmed there's something where the client is signing a date header that is sometimes different than the one the server receives (i.e., it's off by exactly 1 second every time). I'm not sure yet whether the problem is node-smartdc, restify, or node itself yet - this is definitely a new regression though that didn't used to happen. I'll update the ticket again when I have a code fix. m |
Hey Beau, Ok, that took less time to track down than I thought -- can you try using it out of the #v6.5 branch and let me know if it fixes your cases? It should - the problem was that the Date header was getting dropped as the restify client was looking for |
It's done 800 iterations without failing, seems like it's fixed to me! Thanks for the quick work. :) |
Great! On Fri, Mar 29, 2013 at 4:19 PM, Beau Gunderson [email protected]:
|
I'm on Ubuntu 12.10 and hitting this very often which makes scripting the tools a huge pain.
Here's my simple repro:
The text was updated successfully, but these errors were encountered: