CODEWITHSUNDEEP
angulartypescriptasync-awaitangular-promise
kris_IV
kris_IV

Reputation: 2444

Angular2 wait on $http response

I need user ID in almost all REST requests.

When I log-in a user, I pass token to local storage and ID to user.service.ts (by function setUserId(userId);) but when I authenticate user only by token (user is logged but page was refreshed) I try to get userId from User object (get user object from REST request by token).

This part of my code look like:

getUser() {
    var authToken = localStorage.getItem('authorization')

    if(!this.userPromise){
      let headers = new Headers();
      headers.append('Content-type', 'application/json');
      headers.append('Authorization', authToken);

      this.userPromise = this.http.get(
        'http://localhost:8080/api/user',
        { headers }
      ).toPromise()
        .then((res) => res.json());
    }

    return this.userPromise;
  };




getUserId(){
    var $self = this;

    if($self.userId){
      return $self.userId;
    } else {
      $self.getUser().then(function(result){
        $self.setUserId(result.id);
        console.log(result.id);
        return result.id;
      });
    }
  }

When any request ask for user ID i use getUserId() and I check if user Id is defined. If is defined i response with this data, but if not - I want get this data from getUser() function.

I can't solve one problem - this request is async and for example "task" service set userId value as undefined - doesn't wait on this new value.

How can I manage with this problem?

---EDIT

Hire is a request - it doens't wait on response;

 getUserTasks() {
    // return this.userService.getUserId();
    var userId = this.userService.getUserId();

    return this.http.get(
      'http://localhost:8080/api/private/'+userId+'/tasks'
      // 'http://localhost:8080/api/private/1/tasks'
    )
    .toPromise()
      .then((res) => res.json());
  }

Upvotes: 3

Views: 4846

Answers (1)

kris_IV
kris_IV

Reputation: 2444

I found solutions - only one thinks to do is a proper way to use async/await function on getUserTasks():

 async getUserTasks() {
    let userId = await this.userService.getUserId();

    return this.http.get(
      'http://localhost:8080/api/private/'+userId+'/tasks'
    )
    .toPromise()
    .then((res) => res.json());
  }

--- EDIT ---

Finally - if somebody find the same problem I move my changes level up - to authentication process. In this way I have access to userId from any part of my application by services UserService.getUserId();

When user is log-in I have time to put all data to variable, but when i make only authentication i modify isLoggedIn() function to be async/await. If ticket exist i send authentication request to get User object.

If I get user object I return true and put userId to userId services. When I get error (no credentials) I make log-out action in getUser() function.

I think that this is the best solutions - it provide all necessary data and give additional verification process in guard.

async isLoggedIn() {
    var $self = this;

    if(localStorage.getItem("authorization")){
      var user = await $self.getUser();
      this.setUserId(user.id);

      return true;
    } else {
      return false;
    }
  }

Upvotes: 4

Related Questions