Can't exchange request token for access token using 1.0 spec about oauth-php

[GoogleCodeExporter]

Seems to be the same issue as i mentioned here: ConsumerHowTo - Comment by 
fiedler.andre, May 08, 2010

Would really be great if someone fixes this. (and put a sample test 
server/client combination into the repository)

Original comment by [email protected] on 17 May 2010 at 8:01

[GoogleCodeExporter]

I'm checking this right now.

Original comment by brunobg%[email protected] on 18 May 2010 at 1:31

[GoogleCodeExporter]

Can confirm, that bugfix from philfreo is working! Just replace line 1350 (and 
following) in OAuthStoreSQL.php with these:

[code]
 if (isset($options['verifier'])) {
    $verifier = $options['verifier'];

    // 1.0a Compatibility : check token against oauth_verifier
    $this->query('
                UPDATE oauth_server_token
                SET ost_token           = \'%s\',
                    ost_token_secret    = \'%s\',
                    ost_token_type      = \'access\',
                    ost_timestamp       = NOW(),
                    ost_token_ttl       = '.$ttl_sql.'
                WHERE ost_token      = \'%s\'
                  AND ost_token_type = \'request\'
                  AND ost_authorized = 1
                  AND ost_token_ttl  >= NOW()
                  AND ost_verifier = \'%s\'
                ', $new_token, $new_secret, $token, $verifier);
 } else {

    // 1.0
    $this->query('
                UPDATE oauth_server_token
                SET ost_token           = \'%s\',
                    ost_token_secret    = \'%s\',
                    ost_token_type      = \'access\',
                    ost_timestamp       = NOW(),
                    ost_token_ttl       = '.$ttl_sql.'
                WHERE ost_token      = \'%s\'
                  AND ost_token_type = \'request\'
                  AND ost_authorized = 1
                  AND ost_token_ttl  >= NOW()
                ', $new_token, $new_secret, $token);
 }
[/code]

Original comment by [email protected] on 25 May 2010 at 12:09

[GoogleCodeExporter]

Ähm, i have to correct me. Its working better, but now i have to authorize the 
consumer for every request! There 
should one more bug.

Original comment by [email protected] on 25 May 2010 at 6:15

[GoogleCodeExporter]

@philfreo

I´ve checked and fixed the bug you mentioned. Can you test this again? It 
seems there´s one more bug... but didn´t get this by now. :(

Original comment by [email protected] on 9 Jun 2010 at 8:53

• Changed state: Started
• Added labels: Priority-High
• Removed labels: Priority-Medium

[GoogleCodeExporter]

Ok, I tested again. I think this works now! closed :o)

Original comment by [email protected] on 10 Jun 2010 at 8:58

• Changed state: Fixed