Cavemanon/piratepoet
7
Fork 0
Code Pull Requests Actions Releases 16 Activity

- Limits the length of subscriber token to allow for smaller URL data

Browse Source 
objects
This commit is contained in:
Vlad
2016-12-11 11:53:49 -05:00
parent b4da3ecfb3
commit 98c6c29716
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
lib/Models/Subscriber.php
View File

@ -14,8 +14,8 @@ class Subscriber extends Model {
const STATUS_UNSUBSCRIBED = 'unsubscribed';
const STATUS_UNCONFIRMED = 'unconfirmed';
const STATUS_BOUNCED = 'bounced';
const SUBSCRIPTION_LIMIT_COOLDOWN = 60;
const SUBSCRIBER_TOKEN_LENGTH = 6;
function __construct() {
parent::__construct();
@ -154,13 +154,17 @@ class Subscriber extends Model {
static function generateToken($email = null) {
if($email !== null) {
return md5(AUTH_KEY.$email);
return substr(md5(AUTH_KEY . $email), 0, self::SUBSCRIBER_TOKEN_LENGTH);
}
return false;
}
static function verifyToken($email, $token) {
return call_user_func('hash_equals', self::generateToken($email), $token);
return call_user_func(
'hash_equals',
self::generateToken($email),
substr($token, 0, self::SUBSCRIBER_TOKEN_LENGTH)
);
}
static function subscribe($subscriber_data = array(), $segment_ids = array()) {