JessYanCoding / MVPArms

⚔️ A common architecture for Android applications developing based on MVP, integrates many open source projects, to make your developing quicker and easier (一个整合了大量主流开源项目高度可配置化的 Android MVP 快速集成框架).
Apache License 2.0
10.29k stars 2.39k forks source link

GlobalHttpHandlerImpl中的onHttpRequestBefore不被执行,设置Header不起作用 #232

Closed chenyang1900 closed 6 years ago

chenyang1900 commented 6 years ago

Look at here

Environment

Bug Description:

Related Code:

GlobalConfiguration中已经配置了GlobalHttpHandlerImpl ;
  .globalHttpHandler(new GlobalHttpHandlerImpl(context))

GlobalHttpHandlerImpl 中
 @Override
    public Request onHttpRequestBefore(Interceptor.Chain chain, Request request) {
       return chain.request().newBuilder()
                .addHeader("version", AppUtils.getAppVersionName())
                .addHeader("d-os", "android")
                .addHeader("devicemodel", DeviceUtils.getModel()).build();
//        return request;
    }
配置文件中页配置
   <meta-data
            android:name="com.bitkinetic.common.base.http.GlobalConfiguration"
            android:value="ConfigModule" /> 

GlobalHttpHandlerImpl中的onHttpRequestBefore不被执行,设置Header不起作用

Bug Log:

Others:

cnmao commented 6 years ago

请问下你是怎么解决的 ? 我这边在线等 18137503798 这个是我的电话

cnmao commented 6 years ago

@chenyang1900

shinobu59 commented 6 years ago

@chenyang1900

我是这样用的,测试没问题,公共参数和header都能加上

image

topCard commented 5 years ago

我也遇到这个问题了,请问你们后面怎么解决的?

cnmao commented 5 years ago

@topCard 就是上边那样

topCard commented 5 years ago

@topCard 就是上边那样

但是,我这里根本就没有执行onHttpRequestBefore这里,好奇怪

cnmao commented 5 years ago

这个方法你写在哪个类里边了 , 这个类有没有设置到配置信息里边 。

topCard notifications@github.com 于2019年3月11日周一 上午11:16写道:

@topCard https://github.com/topCard 就是上边那样

但是,我这里根本就没有执行onHttpRequestBefore这里,好奇怪

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/JessYanCoding/MVPArms/issues/232#issuecomment-471390821, or mute the thread https://github.com/notifications/unsubscribe-auth/AOg0sdmbJPWdBTWkn4PhTGs4qMjo_MiBks5vVcp3gaJpZM4WMRZV .

topCard commented 5 years ago

这个方法你写在哪个类里边了 , 这个类有没有设置到配置信息里边 。 topCard notifications@github.com 于2019年3月11日周一 上午11:16写道: @topCard https://github.com/topCard 就是上边那样 但是,我这里根本就没有执行onHttpRequestBefore这里,好奇怪 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#232 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AOg0sdmbJPWdBTWkn4PhTGs4qMjo_MiBks5vVcp3gaJpZM4WMRZV . image

image

有配置的,看图片

cnmao commented 5 years ago

这个就尴尬了 , 你确定没有走吗? @topCard

cnmao commented 5 years ago

@topCard GlobalConfiguration 这个在xml 配置了吗?

topCard commented 5 years ago

@topCard GlobalConfiguration 这个在xml 配置了吗? image

确定配置了

cnmao commented 5 years ago

这个我就不知道了 @topCard

topCard commented 5 years ago

这个我就不知道了 @topCard

我找找看,谢谢啦

JessYanCoding commented 5 years ago

因为你在 MyRetrofitConfig 里面 new 了一个新的 okhttp, 你把框架默认的 okhttp 都给替换了,框架的默认配置怎么走?

cnmao commented 5 years ago

懵逼

JessYan notifications@github.com于2019年3月13日 周三10:10写道:

因为你在 MyRetrofitConfig 里面 new 了一个新的 okhttp, 你把框架默认的 okhttp 都给替换了,框架的默认配置怎么走?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/JessYanCoding/MVPArms/issues/232#issuecomment-472251503, or mute the thread https://github.com/notifications/unsubscribe-auth/AOg0sdKP87qnhMnnFoMTQKJU3o1s-daaks5vWF4kgaJpZM4WMRZV .

asd821300801 commented 4 years ago

因为你在MyRetrofitConfig里面new了一个新的okhttp,你把框架放在的okhttp都给替换了,框架的替代配置怎么走?

配合使用RetrofitUrlManager后不得不在MyRetrofitConfig里面new一个新的okhttp,有没有什么办法可将RetrofitUrlManager设置进去,并且GlobalHttpHandlerImpl正常执行?

JessYanCoding commented 4 years ago

@asd821300801 https://github.com/JessYanCoding/MVPArms/blob/master/demo/src/main/java/me/jessyan/mvparms/demo/app/GlobalConfiguration.java#L145