Digest Auth - missing Post request payload

[12bchl]

Difference in behaviour between resty v2 -> v3

Post request body is no longer transmitted for existing sessions

	cl := resty.New()
	cl.SetBaseURL(baseURL)
	cl.SetDigestAuth(username, password)

	r, err = cl.R().Get("/") 
	r, err = cl.R().SetFormData(map[string]string{"A": "B"}).Post("/somepath")

The body (encoding is irrelevant) will be present in v2, or if we comment out first GET request it would be visible in v3.

For existing sessions, using digest auth, POST request body is missing.
Verified with wireshark on client machine, no redirection issues.

Oddly I get the same issue with curl v7.88.1

Client OS: 6.1.0-31-amd64

[jeevatkm]

@12bchl It is not an issue in curl or Resty v3, it is by design. Resty v3 digest should perform the same behaviour as curl. Resty v2 was not doing it appropriately, I have addressed it in v3.
Typically, the first request is made to get the value of the header WWW-Authenticate, for which the body is not required.

[12bchl]

@jeevatkm Thank you. Appreciate your work on this great project.

I have trouble understanding this default behavior though. If I SetDigestAuth this will break my session and I won't be able to send any payload.

r, err = cl.R().SetFormData(foo).Post("bar")

Wouldn't it be better to either:

• provide an error when executing request with payload
• or, only execute digest logic when request fails due to lack of authorization

The nomenclature of "SetDigestAuth" seems misleading if it can't be used to handle the entire session.

curl is a just a cli-tool, not a long-lived client.
I don't think resty's "SetDigestAuth" should be equivalent to curl's "--digest" option

[jeevatkm]

@12bchl Thanks for the appreciation. I'm sorry for the delayed response. I was occupied with my personal stuff.

I have mentioned curl in the conversation to explain the behavior of the initial negotiation request only.
If you have used Resty v2 digest auth; Resty v3 should have the same behavior with many improvements.

If you could provide a code snippet to replicate the issue at my end, it would help me to trace the issue.