common icon indicating copy to clipboard operation
common copied to clipboard

feat: implement support for http digest auth (resolve #352)

Open muety opened this issue 2 years ago • 7 comments

Implements #352. I'd need this in blackbox exporter. Let me know what you think! :-)

Digest auth can be tested against https://httpbin.org/#/Auth.

muety avatar Dec 14 '23 12:12 muety

Not sure about the linting errors, perhaps someone could give me some assistance with that?

muety avatar Dec 14 '23 12:12 muety

Any chances to get this merged? Should I tag someone in here?

muety avatar Dec 20 '23 07:12 muety

I don't see any major issues. I do wonder about if github.com/icholy/digest is the best library option for this. I haven't done any evaluation of what the available options are.

Can you describe why you picked this implementation?

SuperQ avatar Dec 20 '23 10:12 SuperQ

Yeah, I was expecting a bit of discussion around this. I don't know about your requirements for pulling in new dependencies. I was hoping Go's standard lib had an implementation of digest auth, but turns out it doesn't. I picked icholy/digest, because it was the only client implementation I could find.

muety avatar Dec 20 '23 11:12 muety

Is there someone I could tag here who is in charge of merging my PR?

muety avatar Dec 28 '23 17:12 muety

Rebase done, can this be merged now?

muety avatar Jan 05 '24 13:01 muety

After almost a year, I guess this is not gonna be merged then?

muety avatar Oct 01 '24 19:10 muety