Skip to content

Commit

Permalink
Fix: ttl when omitted should set the default value
Browse files Browse the repository at this point in the history
  • Loading branch information
ricardopereira committed Nov 3, 2016
1 parent 4e82209 commit 24d3220
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion Source/ARTTokenParams.m
Original file line number Diff line number Diff line change
Expand Up @@ -153,8 +153,9 @@ - (ARTTokenRequest *)sign:(NSString *)key withNonce:(NSString *)nonce {
NSString *keyName = keyComponents[0];
NSString *keySecret = keyComponents[1];
NSString *clientId = self.clientId ? self.clientId : @"";
NSTimeInterval ttl = self.ttl ? self.ttl : [ARTDefault ttl];

NSString *signText = [NSString stringWithFormat:@"%@\n%lld\n%@\n%@\n%lld\n%@\n", keyName, timeIntervalToMilliseconds(self.ttl), self.capability, clientId, dateToMilliseconds(self.timestamp), nonce];
NSString *signText = [NSString stringWithFormat:@"%@\n%lld\n%@\n%@\n%lld\n%@\n", keyName, timeIntervalToMilliseconds(ttl), self.capability, clientId, dateToMilliseconds(self.timestamp), nonce];
NSString *mac = hmacForDataAndKey([signText dataUsingEncoding:NSUTF8StringEncoding], [keySecret dataUsingEncoding:NSUTF8StringEncoding]);

return [[ARTTokenRequest alloc] initWithTokenParams:self keyName:keyName nonce:nonce mac:mac];
Expand Down
3 changes: 2 additions & 1 deletion Source/ARTTokenRequest.m
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,13 @@
#import "ARTTokenRequest.h"
#import "ARTTokenParams.h"
#import "ARTAuth+Private.h"
#import "ARTDefault.h"

@implementation ARTTokenRequest

- (instancetype)initWithTokenParams:(ARTTokenParams *)tokenParams keyName:(NSString *)keyName nonce:(NSString *)nonce mac:(NSString *)mac {
if (self = [super init]) {
self.ttl = tokenParams.ttl;
self.ttl = tokenParams.ttl ? tokenParams.ttl : [ARTDefault ttl];
self.capability = tokenParams.capability;
self.clientId = tokenParams.clientId;
self.timestamp = tokenParams.timestamp;
Expand Down

0 comments on commit 24d3220

Please sign in to comment.