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

Authentication #32

Open
cpaulbond opened this Issue Mar 16, 2016 · 9 comments

Comments

Projects
None yet
6 participants
@cpaulbond

cpaulbond commented Mar 16, 2016

I check the documentation and did not see any support for HTTP Authentication. Did I miss something or is this an upcoming feature? Also, thanks for a great library!

@ndw

This comment has been minimized.

ndw commented Apr 27, 2016

+1

I'd like to integrate some elisp with a call to a web service that requires digest auth...

@DamienCassou

This comment has been minimized.

DamienCassou commented Nov 4, 2016

I also need that. Currently, I have to do that:

(let* ((url-request-method "GET")
       (url-request-extra-headers
        `(("Content-Type" . "application/json")
          ("Authorization" .
           ,(concat "Basic "
                    (base64-encode-string
                     (concat username ":" password)))))))
  (url-retrieve-synchronously url nil t))
@ahonnecke

This comment has been minimized.

ahonnecke commented Sep 5, 2017

+1

@hkjels

This comment has been minimized.

hkjels commented Jan 5, 2018

:headers `(("Authorization" . ,(concat "Bearer " auth-token)))

Seems to work just fine

@ndw

This comment has been minimized.

ndw commented Jan 7, 2018

With digest auth? Or only with basic auth?

@hkjels

This comment has been minimized.

hkjels commented Jan 8, 2018

I've just tried basic auth

@ndw

This comment has been minimized.

ndw commented Jan 8, 2018

Right. It's specifically digest auth that I need.

@titaniumbones

This comment has been minimized.

Collaborator

titaniumbones commented Nov 29, 2018

@ndw if you can implement this feature (which I don't use and don't know much about) I can merge your solution.

@ndw

This comment has been minimized.

ndw commented Nov 29, 2018

I'll put it on my todo list 😄

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