search

webworkoffice / partakein

Automatically exported from code.google.com/p/partakein
0 stars 0 forks source link

Revoked access authorities are still in datastore. #103

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Twitter users can revoke access authority at below:
http://twitter.com/settings/connections

PARTAKE doesn't delete revoked user account from datastore, so
a lot of '401 Unauthorized: Authentication credentials were missing or 
incorrect.' are returned by Twitter.

Original issue reported on code.google.com by skypencil@gmail.com on 24 Jan 2011 at 3:23

GoogleCodeExporter commented 9 years ago 
Created a patch to separate logging. May I apply this patch to trunk?

Original comment by skypencil@gmail.com on 24 Jan 2011 at 3:48

GoogleCodeExporter commented 9 years ago 
LGTM.

Original comment by kawanaka.shinya@gmail.com on 24 Jan 2011 at 3:54

GoogleCodeExporter commented 9 years ago 
re-attach my patch.

Original comment by skypencil@gmail.com on 24 Jan 2011 at 4:01

Attachments:

GoogleCodeExporter commented 9 years ago 
unauthorized 
なのがあったら、なんかフラグを立てたほうが良いかもし��
�ませんね。フラグが立っていると DM は送られない。再び 
login していただいた場合はフラグ解除。

Original comment by kawanaka.shinya@gmail.com on 24 Jan 2011 at 4:13

GoogleCodeExporter commented 9 years ago 
それがいいですね！再びloginしていただいた場合はTwitterのto
ken/SecretTokenとかも更新する感じで。

Original comment by skypencil@gmail.com on 25 Jan 2011 at 7:13

GoogleCodeExporter commented 9 years ago 
AccessToken (AT) と AccessTokenSecret (ATS) 
を消したらどうかな。人を勝手に invite 
してくる、というのも考えていて、そんときは AT / ATS 
は使えない可能性もある。「連絡ができない」という状態��
� AT / ATS がないとして扱うのは悪くないような気がする。

Original comment by kawanaka.shinya@gmail.com on 26 Jan 2011 at 2:45

GoogleCodeExporter commented 9 years ago 
I wrote a patch for this issue. I'll test for this next weekend.
Please tell me if you find any spelling mistake.

> 人を勝手に invite 
してくる、というのも考えていて、そんときは AT / ATS 
は使えない可能性も
> ある。「連絡ができない」という状態を AT / ATS 
がないとして扱うのは悪くないような気がする。
そういうユーザはそもそもTwitterLinkageがnullまたはNULL_OBJECTで
あるべきかと考えます。
今回Linkageの状態としてUnauthorizedを作ったので、これを使う�
��も悪くないとは思いますが……。

ユーザの状態として「連絡ができない」を持つことを考え��
�もいいと思いますが、それは
TwitterLinkageの役割を考えると好ましくないですかね。

Original comment by skypencil@gmail.com on 2 Mar 2011 at 12:58

Attachments:

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r389.

Original comment by skypencil@gmail.com on 6 Mar 2011 at 7:08

GoogleCodeExporter commented 9 years ago 
Migrated to http://github.com/partakein/partake/issues/103

Original comment by skypencil@gmail.com on 27 Jun 2012 at 9:34