diff options
| author | Evan Prodromou <evan@prodromou.name> | 2008-06-02 16:17:07 -0400 |
|---|---|---|
| committer | Evan Prodromou <evan@prodromou.name> | 2008-06-02 16:17:07 -0400 |
| commit | e103c8b6ea2d06cc15e1f8571936b6a2971fe4fa (patch) | |
| tree | 5bab20f9a50b83a52199ae497673fee0f324b994 /actions/finishremotesubscribe.php | |
| parent | 7b24d101c07ba40b5c8b648bdca0cd6d16ad15f1 (diff) | |
get rid of callback nonce
darcs-hash:20080602201707-84dde-400855b57fcf01e597494143cc78092351043b91.gz
Diffstat (limited to 'actions/finishremotesubscribe.php')
| -rw-r--r-- | actions/finishremotesubscribe.php | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/actions/finishremotesubscribe.php b/actions/finishremotesubscribe.php index b5093263e..41bc91afd 100644 --- a/actions/finishremotesubscribe.php +++ b/actions/finishremotesubscribe.php @@ -33,14 +33,7 @@ class FinishremotesubscribeAction extends Action { return; } - $nonce = $this->trimmed('nonce'); - - if (!$omb) { - common_user_error(_t('No nonce returned!')); - return; - } - - $omb = $_SESSION[$nonce]; + $omb = $_SESSION['oauth_authorization_request']; if (!$omb) { common_user_error(_t('Not expecting this response!')); @@ -173,7 +166,7 @@ class FinishremotesubscribeAction extends Action { } # Clear the data - unset($_SESSION[$nonce]); + unset($_SESSION['oauth_authorization_request']); # If we show subscriptions in reverse chron order, this should # show up close to the top of the page @@ -187,7 +180,7 @@ class FinishremotesubscribeAction extends Action { $con = omb_oauth_consumer(); $tok = new OAuthToken($omb['token'], $omb['secret']); - $url = $omb[OAUTH_ENDPOINT_ACCESS][0]; + $url = omb_service_uri($omb[OAUTH_ENDPOINT_ACCESS]); # XXX: Is this the right thing to do? Strip off GET params and make them # POST params? Seems wrong to me. |