Skip to content
This repository has been archived by the owner on Jul 30, 2018. It is now read-only.

Minor Feature Request : core/request : store redirected URLs #284

Open
sebilasse opened this issue Feb 9, 2017 · 3 comments
Open

Minor Feature Request : core/request : store redirected URLs #284

sebilasse opened this issue Feb 9, 2017 · 3 comments

Comments

@sebilasse
Copy link

It could be useful to store all redirecting URLs in an array e. g. in the options object.
The options are returned with the response and the user now knows the URLs "in between" request and response.

@dylans dylans added this to the 2017.04 milestone Mar 6, 2017
@dylans dylans modified the milestones: 2017.05, 2017.04 Apr 7, 2017
@eheasley eheasley modified the milestones: 2017.05, 2017.06, 2017.07 Jun 6, 2017
@kitsonk kitsonk added the beta3 label Jul 27, 2017
@kitsonk
Copy link
Member

kitsonk commented Jul 27, 2017

It is worth investigating, in that browser caches are likely handling this automatically when a 301 or a 302 is provided. It might be totally unnecessary.

@sebilasse
Copy link
Author

Well, it is not about handling. It is more about knowing.
For example Web-Sign In solutions like RelMeAuth or IndieAuth :
It would be useful to say until the end of the session that all these URLs redirect to target X.

@kitsonk
Copy link
Member

kitsonk commented Jul 28, 2017

That really feels like a higher order application concern. If you are in possession of a token that means you are logged in, then remapping a load of URLs is something an application can do. Worst case, the approach would be some sort of request provider that is then registered as the default provider (or even a particular set of URLs) with something like:

import { providerRegistry } from '@dojo/core/request';

providerRegistry.register(/some.sort.of.test/, myAwesomeProvider, true);

@kitsonk kitsonk removed the beta3 label Oct 30, 2017
@kitsonk kitsonk removed this from the long-grass milestone Oct 30, 2017
@kitsonk kitsonk added the rc label Oct 30, 2017
@kitsonk kitsonk added this to the long-grass milestone Oct 30, 2017
@agubler agubler removed the rc label Jan 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants