#99 Added ability to include more storage options for authentication #100

Merged
merged 33 commits into from Dec 15, 2014

Conversation

Projects
None yet
3 participants
Contributor

reklov commented Dec 3, 2013

We refactored the current storage solution, as it is limited to file storage only. It now

a) provides several adapters to use either file storage or redis (added ability to add more in future) for authentication
b) does not use instance variables from other classes, uses methods instead

Added specs for all refactored code. Old FileStorage Class has been refactored to use the new Adapters, you might want to mark it as deprecated. Cleaned gemspec to not include all files manually, if you want to exclude some files, you can use the exclude flag for this.

This code has been tested to work with your drive examples and does not break.

reklov and others added some commits Nov 14, 2013

ignore IntelliJ files
ignore logfiles
refactoring load_credentials
adding refresh_authorization

write_credentials now using the authorizations methods not the instance variables, Using the instance variables can cause problems.
adds Constants for Authorization_uri, Token_credentials_uri
extracts authorize
extracts credentials_hash
use authorization method instead of variable
remove authorize from initialize
adds specs for storage
changes expectation in write_credentials

@sqrrrl sqrrrl merged commit fd145d2 into google:master Dec 15, 2014

1 check failed

default The Travis CI build could not complete due to an error
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment