I'm using a http nested call in angular.
First Call to get a token. Second call will be using token and returning actual data.
public get(api) {
return this.http.get(this.url + '/token')
.map(res => res.json())
.toPromise()
.then((data: any) => {
this.access_token = data.access_token
let headers = new Headers();
headers.append('Authorization', 'Bearer ' + this.access_token);
return this.http.get(this.url + api, {
headers: headers
}).map(res => res.json());
})
}
The function returns Promise<Observable<any>>
. How to parse the response so i can get the data from nested call in a subscription.
this.get('me/profile')
.subscribe(data => {
console.log(data);
});
I'm using a http nested call in angular.
First Call to get a token. Second call will be using token and returning actual data.
public get(api) {
return this.http.get(this.url + '/token')
.map(res => res.json())
.toPromise()
.then((data: any) => {
this.access_token = data.access_token
let headers = new Headers();
headers.append('Authorization', 'Bearer ' + this.access_token);
return this.http.get(this.url + api, {
headers: headers
}).map(res => res.json());
})
}
The function returns Promise<Observable<any>>
. How to parse the response so i can get the data from nested call in a subscription.
this.get('me/profile')
.subscribe(data => {
console.log(data);
});
Share
Improve this question
edited Aug 28, 2017 at 20:42
Alexander Staroselsky
38.9k16 gold badges82 silver badges98 bronze badges
asked Aug 28, 2017 at 20:23
Sumit RidhalSumit Ridhal
1,4193 gold badges15 silver badges30 bronze badges
3
-
1
Try removing the
.toPromise()
. That should return anObservable<any>
. I remend you to use type (Class
) to map data so the results are object rather than JSON strings. Like Java. – Prav Commented Aug 28, 2017 at 20:27 - 2 Either be consistent with promises (return a promise from then callback) or observables (flatmap over the second call). Mixing is a recipe for disaster. – jonrsharpe Commented Aug 28, 2017 at 20:28
- You're mixing two different methods of handling async data, which is hellish in it's own right. Either produce Promise Chaining to get the data, use .then or switch away from the conversion to a promise being returned in your get function. – L.P. Commented Aug 28, 2017 at 20:29
1 Answer
Reset to default 6You could use RxJS operators such as switchMap to streamline this nested call and without using toPromise()
.
Map to observable, plete previous inner observable, emit values.
You are effectively passing emitted values of first observable to the next observable (your second HTTP call). This allows you subscribe to the final results in your usage example.
It is important that within the switchMap()
you return
the inner observable, in this case the HTTP call for this to function effectively.
As others have indicated, toPromise()
is not necessary as you can take advantage of RxJS operators to plete this async, dependent action.
public get(api) {
return this.http.get(this.url + '/token')
.map(res => res.json())
.switchMap(data => {
this.access_token = data.access_token;
let headers = new Headers();
headers.append('Authorization', 'Bearer ' + this.access_token);
return this.http.get(this.url + api, { headers: headers });
})
.map(res => res.json());
}
usage
this.get('me/profile').subscribe(data => console.log(data));
Note: With Angular 4+ HttpClient, you don't need to explicitly call json()
on the response as it does it automatically.