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

Twitter verification doesn't work #47

Closed
aaronpk opened this Issue Dec 3, 2015 · 4 comments

Comments

3 participants
@aaronpk
Member

aaronpk commented Dec 3, 2015

Is most likely the same problem as aaronpk/IndieAuth.com#108

@0xdevalias

This comment has been minimized.

Show comment
Hide comment
@0xdevalias

0xdevalias Apr 12, 2016

+1 having this issue

+1 having this issue

@aaronpk

This comment has been minimized.

Show comment
Hide comment
@aaronpk

aaronpk Aug 2, 2016

Member

It looks like this is actually because Twitter is not sending an HTTP redirect for browser user agents. Instead they send an HTTP 200 response and an HTML redirect instead. We can probably easily fix this by not sending a browser-like user agent when fetching t.co URLs, or add support for handing HTML meta redirect responses as well.

Member

aaronpk commented Aug 2, 2016

It looks like this is actually because Twitter is not sending an HTTP redirect for browser user agents. Instead they send an HTTP 200 response and an HTML redirect instead. We can probably easily fix this by not sending a browser-like user agent when fetching t.co URLs, or add support for handing HTML meta redirect responses as well.

@gRegorLove

This comment has been minimized.

Show comment
Hide comment
@gRegorLove

gRegorLove Aug 2, 2016

Member

I think that fetching takes place in the rel-me lib https://github.com/indieweb/rel-me

Also discussion reference: https://chat.indieweb.org/dev/2016-07-26#t1469574949249000

Member

gRegorLove commented Aug 2, 2016

I think that fetching takes place in the rel-me lib https://github.com/indieweb/rel-me

Also discussion reference: https://chat.indieweb.org/dev/2016-07-26#t1469574949249000

@gRegorLove

This comment has been minimized.

Show comment
Hide comment
@gRegorLove

gRegorLove Sep 8, 2016

Member

Closing since more context captured and soon to be resolved in #52.

Member

gRegorLove commented Sep 8, 2016

Closing since more context captured and soon to be resolved in #52.

@gRegorLove gRegorLove closed this Sep 8, 2016

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