Comments (4)
it does not seem to work when I original request is http and I get 301 to https, probably because of
"HttpURLConnection will follow up to five HTTP redirects. It will follow redirects from one origin server to another. This implementation doesn't follow redirects from HTTPS to HTTP or vice versa."
from volley.
Which HTTP stack are you using?
If you're using HurlStack, I think you can just call HttpUrlConnection.setFollowRedirects(true).
If you're using HttpClientStack for some reason, consider not using it, but you would probably still be okay if you customized the AndroidHttpClient being used under the covers to undo the default call to HttpClientParams.setRedirecting(params, false).
For other stacks, I think it feels more natural to me that the HTTP stack handles redirects internally rather than have them bubbled up to the Volley network layer and require that they be accounted for in retry policies. LMK if that doesn't make sense and I can take a closer look.
from volley.
Got it - thanks for the pointer.
Interestingly okhttp's stack changes that default behavior (and notes as much):
So I'm still not 100% sure whether it would make more sense to do this at the HTTP stack layer or at the Volley layer. Either way, though, we should make this possible in some form.
from volley.
Having investigated this a bit more carefully, I don't think this is a change I'd be comfortable making to Volley, unfortunately.
Following redirects correctly in all cases is really tricky: have a look at OkHttp's code to do so. I don't think it's wise for Volley to try and imitate this logic; Volley's intent is to provide a light interface on top of network layers which is where this logic belongs.
If HttpUrlConnection supported a flag to enable this behavior, it'd certainly be reasonable to expose an API in HurlStack to access it if needed, but it does not.
So, my suggestions would be to either:
- Avoid encountering this scenario by making requests using the desired post-redirect protocol, be it HTTP or HTTPS, so that redirects don't change protocols
- Use a different HTTP stack implementation with Volley that supports this innately. There are various OkHttp-backed stacks floating around, for example. (The Apache stack might support them, though I wouldn't recommend this as it's deprecated per the Android platform, and I haven't tested).
- Extend HurlStack in your own app and override executeRequest to implement the redirect handling as desired. As noted above, it's tricky to do this generically, but if you know the backends you are hitting and the redirect scenarios you want to support then you might be able to cover a minimal set of reasonable cases more easily.
from volley.
Related Issues (20)
- The JsonObjectRequest and JsonArrayRequest don't support 204: No Content HOT 5
- Unexpected response code 511 for HOT 1
- Support digest authentication HOT 2
- Type com.android.volley.Cache$Entry is defined multiple times HOT 1
- Caused by: com.android.tools.r8.utils.AbortException: Error: Program type already present: com.google.android.youtube.player.YouTubeApiServiceUtil HOT 1
- Volley unable to parse dateStr HOT 1
- setPriority() for Request HOT 1
- E/Volley: [18667] NetworkUtility.shouldRetryException: Unexpected response code 403 HOT 1
- -
- Com.android.volley.2015.05.28.jar HOT 1
- Support Proxy functionality HOT 1
- Volley: [5311] NetworkUtility.logSlowRequests: HTTP response for request=<[ ] HOT 3
- NetworkUtility.logSlowRequests: HTTP response for request=<[ ] https://www.google.com/ 0xd61ac103 NORMAL 1> [lifetime=6472], [size=159525], [rc=200], [retryCount=1]
- Fix nullability issues in Kotlin sample code HOT 4
- Pagination HOT 1
- Getting E/Volley: [19618] NetworkDispatcher.processRequest: Unhandled exception HOT 4
- pls rply me what can i do for this HOT 1
- Potential debug logging even when DEBUG is False HOT 6
- ConcurrentModificationException on cancel HOT 1
- App Crashed on Android version 4.3 java.lang.NoClassDefFoundError: com.android.volley.toolbox.Volley HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from volley.