fasheng / elfeed-protocol

Provide extra protocols to make like Fever, NewsBlur, Nextcloud/ownCloud News and Tiny Tiny RSS work with elfeed
GNU General Public License v3.0
97 stars 18 forks source link

elfeed protocol fails to connect with bazqux.com with fever api #57

Closed ghost closed 1 year ago

ghost commented 1 year ago

elfeed-log

[2023-04-08 18:58:05] [debug]: elfeed-protocol-fever: {"api_version":3,"auth":0} [2023-04-08 18:58:05] [error]: elfeed-protocol-fever: authentication failed, wrong username or password

bazqux.com has a fever api which works with simple curl authentication. It has the following format

user: email@somedomain password: somepassword

An md5sum of user:password works fine with curl authentication, but only using the email as username.

The same fails in using elfeed protocol

    (setq elfeed-feeds (list
                        (list "fever+https://user@somedomain@bazqux.com"
                              :api-url "https://bazqux.com/fever/"
                              :password "somepassword")))
fasheng commented 1 year ago

I see, so your username contains @ and url-generic-parse-url could not parse it correctly. I will try to solved it, later.

(url-generic-parse-url "fever+https://user@somedomain@bazqux.com")

https://github.com/fasheng/elfeed-protocol/blob/master/elfeed-protocol-common.el#L139

fasheng commented 1 year ago

@pirattidasan I give a workaround for such issue, please test if working for bazqux.com.

ghost commented 1 year ago

Works great ! Thank you.