search

jimdoescode / CodeIgniter-Dropbox-API-Library

This library is written for the CodeIgniter PHP framework and is meant to interact with the Dropbox API
http://jimdoescode.blogspot.com/2011/07/codeigniter-dropbox-api.html
154 stars 56 forks source link

Store in db? Method missing in docs #4

Closed talkingnews closed 12 years ago

talkingnews commented 12 years ago

Hi Jim,

Great to see you updating this library.

In your docs, you mention:

$oauth = $this->dropbox->get_access_token($this->session->userdata('token_secret')); $this->_store_in_db($oauth['oauth_token']); $this->_store_in_db($oauth['oauth_token_secret']);

Which, of course, gives me

Call to undefined method Dbox::_store_in_db()

Is this pseudocode, or have I skipped a section of the docs, or am I missing a library or a dependency or something?

My use for your library is to authenticate (once) as me, then when users upload files, then get copied to my dropbox and I get the sharing link back, so it's me authenticated, not them. Hence the db (database not dropbox) storage method.

Over at https://github.com/BenTheDesigner/Dropbox/tree/master/Dropbox/OAuth/Storage he's got a db encrypted storage method, and warns to never store tokens unencrypted.

Does your library have anything equivalent, or might it be possible to make a hybrid including some of his code? I'm slightly into "over my head" territory here so pointers would be appreciated!

jimdoescode commented 12 years ago

The store in db method is pseudocode to demonstrate what you might want to do with the token data. It is purely for example purposes and was not meant to be implemented. Sorry for the confusion.

In your case you will want to record the token data in a config file and recall it that way. You only need to store the token data in a database if you are authenticating with multiple users and those users have database entries.

I am closing this ticket as it's not an issue.

talkingnews commented 12 years ago

OK, I understand. As it happens, I just found that the CMS I am using, PyroCMS, actually has a built-in encrypted field type in its ORM, so in fact it's as easy as (something along the lines of)

$entry_data = array( 'oauth_token' => $oauth['oauth_token'], 'oauth_token_secret' => $oauth['oauth_token_secret'] ); $this->streams->entries->update_entry(1, $entry_data, 'tokens', 'streams');

and retrieve with

$token = $this->streams->entries->get_entry(1, 'tokens', 'streams');

as I found about 5 minutes before you posted here. Working well now, great library, thanks again.