Skip to content
This repository was archived by the owner on Feb 16, 2021. It is now read-only.

2.0.0 #19

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

2.0.0 #19

wants to merge 8 commits into from

Conversation

thekalinga
Copy link

@thekalinga thekalinga commented Jan 19, 2017

I done a major refactoring that includes the following new features

  1. Ability to handle errors properly
  2. Ability for interceptors (in request, response, error, short circuit flows) complete the flow by skipping over the rest of the interceptor chain
  3. Ability to interceptors short circuit the flow, so that the response flow kicks in starting with the interceptor that asked for the short circuit
  4. Ability to perform retries on the real http.request(..) response observable; eg. usecase is OAuth2 flow

Please note that this is not a completed tested code.

Please review the changes & share your views

import { InterceptorUtils } from './interceptor-utils';
import { RealResponseObservableTransformer } from './real-response-observable-transformer';

import './object-assign-polyfill';
Copy link
Author

@thekalinga thekalinga Jan 19, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This import has to be removed from the code. Dint see any errors by tsc. Forgot to remove

@thekalinga
Copy link
Author

Once the changes are finalised, we can create a branch angular 4 for angular 4 version.

The artifact name for that branch can be ng4-http-interceptor

…Direct` which gives access to http API bypassing interception
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant