Skip to content

Update digest_auth.rb#11

Open
i-rus wants to merge 1 commit into
drbrain:masterfrom
i-rus:master
Open

Update digest_auth.rb#11
i-rus wants to merge 1 commit into
drbrain:masterfrom
i-rus:master

Conversation

@i-rus

@i-rus i-rus commented Nov 16, 2013

Copy link
Copy Markdown

The nonce_count for the first authorization request should be 1. Currently 0 is used, which fails causes the authentication to fail with some servers (e.g. Node.js). This changes the initial nonce_count value to 0 so that the value used for the first request will be 1.

The nonce_count for the first authorization request should be 1. Currently 0 is used, which fails causes the authentication to fail with some servers (e.g. Node.js). This changes the initial nonce_count value to 0 so that the value used for the first request will be 1.
@drbrain

drbrain commented Nov 18, 2013

Copy link
Copy Markdown
Owner

Looks good, but can you update the tests too?

It looks like you'll need to adjust the nc and response values. Copying what's in the travis output should get things working so you won't need to run the tests locally. When you push your commit, travis will auto-run to make sure.

Here's the failures:

https://travis-ci.org/drbrain/net-http-digest_auth/jobs/14077840#L86

@paulvt

paulvt commented Mar 21, 2018

Copy link
Copy Markdown

Justa added pull request #17 that supersedes this one. This was an accidental duplicate, sorry!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants