-
Notifications
You must be signed in to change notification settings - Fork 12
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
handle connection failures to token validation endpoint #80
Comments
hey xavier, sorry for the trouble! looks like this response returned 200 OK, and those are just log messages that didn't actually cause an error. i'm guessing posting with quill itself still worked ok? if so, this is probably the same as #50. worthwhile, but not high priority. |
No. The post did not get through. |
ok! sorry about that. looks like the key part is |
I had tried that already. My webhosters say that they aren't stripping the headers. |
sorry, you're right. looks like it's getting the token from the POST body, not from the header. hey @aaronpk, https://indieauth.com/developers implies that the verification endpoint always returns a response body, even for errors. is that true? |
friendly ping @aaronpk! |
This is actually hitting
|
No @aaronpk. This issue still exists. I just tried out a test post from Quill and this is what I got: HTTP/1.1 200 OK Date: Sun, 31 Dec 2017 16:58:32 GMT Server: Apache Link: https://tokens.indieauth.com/token; rel="token_endpoint", http://xavierroy.com/wp-json/webmention/1.0/endpoint; rel="webmention", http://xavierroy.com/wp-json/webmention/1.0/endpoint; rel="http://webmention.org/" Cache-Control: max-age=3600 Expires: Sun, 31 Dec 2017 17:58:32 GMT Vary: Accept-Encoding Transfer-Encoding: chunked Content-Type: application/json; charset=UTF-8 Notice: Undefined index: authorization in /hsphere/local/home/xavier/xavierroy.com/wp-content/plugins/micropub/micropub.php on line 939 Notice: Undefined index: scope in /hsphere/local/home/xavier/xavierroy.com/wp-content/plugins/micropub/micropub.php on line 173 "invalid access token: " |
hmm I don't see a request to the token endpoint at that timestamp. That indicates to me that there's a problem with the Wordpress server hitting |
For the record, we did a little troubleshooting in chat. Installing this PHP script on the server helped give a little more insight into the problem. https://gist.github.com/aaronpk/001a5c91c823017241e8e2f66da7dd07 The specific curl error we were seeing was:
It might be helpful if this plugin could help surface those errors better instead of just silently failing. This is a problem at the host level, which is using a super old version of OpenSSL and/or cURL, which first didn't support the cipher list in use, and also isn't sending the SNI header for the connection. There unfortunately isn't a good solution I can do on my end of |
thanks for the sleuthing and reporting! and absolutely right, we should surface these errors to the micropub client. will do. |
When I tried using Quill after enabling Micropub, I get these errors:
HTTP/1.1 200 OK
Date: Fri, 24 Nov 2017 09:33:05 GMT
Server: Apache
Link: https://tokens.indieauth.com/token; rel="token_endpoint"
Cache-Control: max-age=3600
Expires: Fri, 24 Nov 2017 10:33:05 GMT
Vary: Accept-Encoding
Transfer-Encoding: chunked
Content-Type: application/json
Notice: Undefined index: authorization in /hsphere/local/home/xavier/xavierroy.com/wp-content/plugins/micropub/micropub.php on line 968
Notice: Undefined index: scope in /hsphere/local/home/xavier/xavierroy.com/wp-content/plugins/micropub/micropub.php on line 159
"invalid access token: "
I am using the unreleased 1.3 version.
The text was updated successfully, but these errors were encountered: