Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow caching of authorization request #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Wouter33
Copy link

This pull requests adds a generic parameter to the construct option to provide an existing authorization object and allows you to get the authorization details from an existing connection. This way it is possible by the user to implement their own caching mechanism.

Example usage with Redis caching could be:

// Check if we have an authorization object available in Redis
$redis = \Redis::getInstance();
$auth = $redis->get('b2_auth');

if(!empty($auth)){
	
	// If available, decode object and use to init library
	$auth = json_decode($auth, true);
	$this->client = new \ChrisWhite\B2\Client(B2_ID, B2_KEY, ['authorization' => $auth]);
	
} else {
	
	// If not, generate init with new authorization request and save to Redis afterwards
	$this->client = new \ChrisWhite\B2\Client(B2_ID, B2_KEY);
	$authorization = $this->client->getAuthorization();
	
        // Save authorization object to Redis, expire in 23 hours so we don't use any expired tokens
	$redis->set('b2_auth', json_encode($authorization, JSON_UNESCAPED_SLASHES));
	$redis->expire('b2_auth', 60 * 60 * 23);	
	
}

@iammichiel
Copy link

iammichiel commented Jun 22, 2018

I do not know what to think of this. I understand the need, however, the implementation bothers me.
With the current deisgn, you have to provide the tokens from outside the library, meaning you have to do the authorization somewhere outside of the library as well.

@Wouter33
Copy link
Author

Wouter33 commented Jun 22, 2018

@iammichiel I choose explicitly for the current design as to make it as generic as possible so everybody can choose their own caching method.

Authorization doesn't happen outside the library, it allows you to retrieve the authorization token after the initial connection by the library and insert it into the library again at a later time.

@OscarEriksen
Copy link

I'm with @iammichiel here. Perhaps provide a cache adapter that can be injected in to the library?

I still think and agree that this is a nice quickfix @Wouter33. My solution is quite similar.

@Wouter33
Copy link
Author

@OscarEriksen A cache adapter plugin possibility and the cache adapters themselves require a much bigger overhaul and limit the freedom to use whatever the user want. I agree that that would be better and the ideal situation, but looking at this repo's activity and actual contributions i doubt how realistic it is that we will be getting that in the near future. At least this pull request will allow some sort of caching by users without forking the repo.

@OscarEriksen
Copy link

I agree with you. It's a nice quickfix and opens up for further development. @cwhite92 input?

If anyone wants to create a fork (seeing as the current maintainer doesn't really have time, no hard feelings) and work on an up-to-date php-library for Backblaze where the aim is to cover issues such as this, then I'd like to join in. And I think Backblaze would be greatly appreciative of it as well.

@tarikozket
Copy link

I forked the repo and merging the PRs in there. You can access it here: https://github.com/tarikozket/b2-sdk-php

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants