SDWebImage 链接重定向问题处理

Round 1

最近公司的文件服务器进行了改造,即使是图片的加载请求也要携带token,否则无法加载,而我们项目中图片加载用的是SDWebImage,当时听到这个需求我内心毫无波动,心里已经....你懂得,不过该做还是要做,三下五除二写完了代码如下:

[[SDWebImageDownloader sharedDownloader] setValue:@"你的token" forHTTPHeaderField:@"Authorization"];

SDWebImage的下载处理是由SDWebImageDownloader单例类实现,所以在你项目中合适的地方加上这句代码,项目中所有用SDWebImage做图片加载的地方就都会携带上token了

Round 2

这样修改完后确实本来不能加载的地方变得正常了,直到那一天,那是一个春天... 项目中要添加一个需求,需要引用公司的一个私有Pod功能库,又是一顿操作,集成完毕,逻辑编写完成,run,诶,图片居然没加载出来,我草这什么情况,我再次确认了一下,我的token设置完成了的
我去询问编写这个Pod的同事,是不是我哪里没配置完成,他略微沉思一下两秒钟后说,你添加完token还要在SDWebImageDownloader修改下源码,我:???,随后他找到了这个源码,代码如下:

- (void)URLSession:(NSURLSession *)session task:(NSURLSessionTask *)task willPerformHTTPRedirection:(NSHTTPURLResponse *)response newRequest:(NSURLRequest *)request completionHandler:(void (^)(NSURLRequest * _Nullable))completionHandler {

    // Identify the operation that runs this task and pass it the delegate method
    NSOperation<SDWebImageDownloaderOperation> *dataOperation = [self operationWithTask:task];
    if ([dataOperation respondsToSelector:@selector(URLSession:task:willPerformHTTPRedirection:newRequest:completionHandler:)]) {
        [dataOperation URLSession:session task:task willPerformHTTPRedirection:response newRequest:request completionHandler:completionHandler];
    } else {
        if (completionHandler) {
            completionHandler(request);
        }
    }
}

他解释说道,这个功能模块里的一些图片链接中携带了一些参数,并不是直接指向资源,所以请求会进行重定向,所以需要在这里进行处理,处理后的代码如下:

- (void)URLSession:(NSURLSession *)session task:(NSURLSessionTask *)task willPerformHTTPRedirection:(NSHTTPURLResponse *)response newRequest:(NSURLRequest *)request completionHandler:(void (^)(NSURLRequest * _Nullable))completionHandler {
    
    // Identify the operation that runs this task and pass it the delegate method
    NSOperation<SDWebImageDownloaderOperation> *dataOperation = [self operationWithTask:task];
    NSMutableURLRequest *customRequest = [[NSMutableURLRequest alloc] initWithURL:request.URL];
    customRequest.allHTTPHeaderFields = self.HTTPHeaders;
    if ([dataOperation respondsToSelector:@selector(URLSession:task:willPerformHTTPRedirection:newRequest:completionHandler:)]) {
        [dataOperation URLSession:session task:task willPerformHTTPRedirection:response newRequest:customRequest completionHandler:completionHandler];
    } else {
        if (completionHandler) {
            completionHandler(customRequest);
        }
    }
}

我将代码修改后,run,确实,问题解决了,但是不对啊,我们的SDWebImage是通过Pod的方式集成的,这样直接在Pod文件夹下修改三方的源码,那么下次更新后,岂不是被覆盖了?这是一个新的问题,于是我开始思考怎么解决重定向问题的同时不修改三方库的源码,脑海中瞬间就想到了AOP
iOS开发中优秀的AOP库那必须有Aspects的名字,接下来我开始思考具体步骤 首先,通过同事提供的解决问题的代码来看,他是把参数request给改为了一个自定义的customRequest,这两个的区别,然后重新设置了allHTTPHeaderFields

NSMutableURLRequest *customRequest = [[NSMutableURLRequest alloc] initWithURL:request.URL];
customRequest.allHTTPHeaderFields = self.HTTPHeaders;

那么我想,问题主要就是在allHTTPHeaderFields这里了,我打印了requestcustomRequestallHTTPHeaderFields后发现,前者比后者少了token,怪不得无法加载,这里要提一下下边这个方法

- (void)URLSession:(NSURLSession *)session task:(NSURLSessionTask *)task willPerformHTTPRedirection:(NSHTTPURLResponse *)response newRequest:(NSURLRequest *)request completionHandler:(void (^)(NSURLRequest * _Nullable))completionHandler 

这其实不是SDWebImageDownloader的方法,是NSURLSessionTaskDelegate的里的协议方法,SDWebImageDownloader实现协议方法后在里边做了自己的重定向处理
那么说一下我一开始的想法,既然问题出在重定向时request里未携带我们手动添加的token,并且重定向这里肯定是要做处理的,那我们直接把相关参数设置给request,没必要创建一个新的customRequest实例

[request setValue:self.HTTPHeaders forKey:@"allHTTPHeaderFields"];

因为request是一个NSURLRequest对象,它的allHTTPHeaderFields是一个readonly属性,我们不能直接修改,所以我想当然的用KVC去操作, 然后 run ,然后 我草crash了,报错信息如下

"[<NSURLRequest 0x2800efa50> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key 
allHTTPHeaderFields."

看描述信息是说NSURLRequest没有对应的allHTTPHeaderFields这个key,有那么一瞬间我愣了下,这不科学啊怎么可能没有,我点进NSURLRequest类确认了下,有啊,什么情况,但是本着求知的态度,我心想是不是NSURLRequest内部使用的是不是不叫allHTTPHeaderFields,但是不对啊,这个属性明明在的啊,即使是别的也应该是内部重新赋值我这里不应该报错啊,不过我还是用通过runtime将他的属性列表打印了一下,再次确认了,他确实有allHTTPHeaderFields这个属性,于是我搜索了下相关问题,发现了一个关键词

+ (BOOL)accessInstanceVariablesDirectly

详细信息自行检索,我这里说下结果,这个是针对KVC的,总的来说,当一个类实现了这个方法并且返回了YES他就可以通过KVC(这个说法不完全准确,因为本文不是针对KVC的故简要说明)赋值,如果返回NO就不可以用KVC赋值
看到这里后我猜测NSURLRequest里这个方法应该是返回了NO,那完了,走不通了,还是要实例化一个新的对象

Round 3

搞了半天想省事看来不行啊,那拉倒,直接开始AOP修改:

    NSError * error ;
    [[SDWebImageDownloader sharedDownloader] aspect_hookSelector:@selector(URLSession: task: willPerformHTTPRedirection: newRequest: completionHandler:) withOptions:AspectPositionInstead usingBlock:^(id<AspectInfo> aspectInfo, NSString *num){
        
        NSArray * param = aspectInfo.arguments;
        NSURLRequest * request = param[3];
        NSURLSessionTask * task = param[1];
        NSMutableURLRequest *customRequest = [[NSMutableURLRequest alloc] initWithURL:request.URL];
        customRequest.allHTTPHeaderFields = task.currentRequest.allHTTPHeaderFields;
        request = customRequest;
        NSInvocation * invocation = aspectInfo.originalInvocation;
        [invocation setArgument:&request atIndex:5];
        [invocation invoke];
        
    } error:&error];

这里边aspectInfo就是被hook方法的信息,可以通过它取到原方法的参数,调用对象等等,我们这里要添加我们的token,因此取出参数进行修改
arguments是原方法的入参列表,是一个数组
invocation是一个消息对象,包含了一个方法的所有信息
通过URLSession: task: willPerformHTTPRedirection: newRequest: completionHandler:方法我们可以知道request的索引是3,task的索引是1(取出它是因为我们要获取原header信息,这个不能丢弃),之后对request重新进行赋值,完成修改,然后重新调用方法

 [invocation setArgument:&request atIndex:5];
 [invocation invoke];

因为我们只需要修改request一个对象,因此只重新设置这一个方法入参,至于这里为什么在赋值的时候索引是5,因为前两个分别被该方法的self_cmd占用,所以我们设置参数的时候索引是从2开始
再次run,嗯,图片顺利加载,问题解决。
这样一来,我们就不需要修改三方的源码就解决了问题,否则修改源码的话每次更新Pod我们的修改就会被覆盖掉,如果哪次发版没注意,测试也没回归覆盖,很容易将问题带到线上

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 206,839评论 6 482
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 88,543评论 2 382
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 153,116评论 0 344
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 55,371评论 1 279
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 64,384评论 5 374
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 49,111评论 1 285
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,416评论 3 400
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 37,053评论 0 259
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 43,558评论 1 300
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,007评论 2 325
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,117评论 1 334
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,756评论 4 324
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,324评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,315评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,539评论 1 262
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,578评论 2 355
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,877评论 2 345

推荐阅读更多精彩内容