Issue
I have an Ionic app that has a user
provider with a signup()
method:
doSignup() {
// set login to same as email
this.account.login = this.account.email;
// Attempt to login in through our User service
this.user.signup(this.account).subscribe((resp) => {
this.navCtrl.push(MainPage);
}, (err) => {
//console.log('error in signup', err);
// ^^ results in 'You provided 'undefined' where a stream was expected'
//this.navCtrl.push(MainPage);
// Unable to sign up
let toast = this.toastCtrl.create({
message: this.signupErrorString,
duration: 3000,
position: 'top'
});
toast.present();
});
}
For some reason, this code never calls the success callback, only the error handler. When it does, it results in the error you see in the comment above.
My user.signup()
method looks as follows:
signup(accountInfo: any) {
return this.api.post('register', accountInfo).share();
}
My Api
class looks as follows:
import { HttpClient, HttpParams } from '@angular/common/http';
import { Injectable } from '@angular/core';
/**
* Api is a generic REST Api handler. Set your API url first.
*/
@Injectable()
export class Api {
public static API_URL: string = 'http://localhost:8080/api';
constructor(public http: HttpClient) {
}
get(endpoint: string, params?: any, reqOpts?: any) {
if (!reqOpts) {
reqOpts = {
params: new HttpParams()
};
}
// Support easy query params for GET requests
if (params) {
reqOpts.params = new HttpParams();
for (let k in params) {
reqOpts.params.set(k, params[k]);
}
}
return this.http.get(Api.API_URL + '/' + endpoint, reqOpts);
}
post(endpoint: string, body: any, reqOpts?: any) {
return this.http.post(Api.API_URL + '/' + endpoint, body, reqOpts);
}
put(endpoint: string, body: any, reqOpts?: any) {
return this.http.put(Api.API_URL + '/' + endpoint, body, reqOpts);
}
delete(endpoint: string, reqOpts?: any) {
return this.http.delete(Api.API_URL + '/' + endpoint, reqOpts);
}
patch(endpoint: string, body: any, reqOpts?: any) {
return this.http.put(Api.API_URL + '/' + endpoint, body, reqOpts);
}
}
I tried removing share()
from user.signup()
, and returning Observable<any>
, but that doesn't help.
Solution
I faced this issue when creating a new project using generator-jhipster-ionic (yo jhipster-ionic giving v3.1.2), following Matt Raible's (OP) Use Ionic for JHipster to Create Mobile Apps with OIDC Authentication blog article, but choosing JWT authentication instead of OIDC.
The origin of the issue was a mix of problems.
I had the issue when running a Ionic app with the livereload server, CORS issues happening and Angular HTTP returning the classic HTTP failure response for (unknown url): 0 Unknown Error
, where 0 is the HTTP error code.
However, in the current case the issue was hidden by bad error handling at the Observable level.
When you follow Angular's HttpClient #Error Details section advices, and add an Observable pipe with a catchError operator on the HTTP POST call, you can get the proper HTTP error details. In this case, instead of going down to the rxjs/util/subscribeToResult.js:71
(TS source #L80) TypeError: You provided 'undefined' where a stream was expected
default error case, it goes to rx/util/subscribeToResult.js:23
(TS source #L34), and the error is handled properly in the piped method.
After following the Observable calls, I found that the current default authentication interceptor, as seen in this template src/providers/auth/auth-interceptor.ts catches HTTP error 401 and does nothing for the others, basically muting them and preventing their propagation.
TL;DR In the JWT case, the solution is to simply remove the src/providers/auth/auth-interceptor.ts
.catch(...)
block, allowing error propagation to login.service.ts
, and in its this.authServerProvider.login(credentials).subscribe((data) => { ... }, (err) => { ... })
error callback.
I believe the issue and solution could be the same for your OIDC case, its signup method, and error callback.
[Edit] Even more since the same .catch
code can be found in the starter example mentioned in the first post comments: ionic-jhipster-starter - auth-interceptor.ts#L31
Answered By - Stéphane Seyvoz
0 comments:
Post a Comment
Note: Only a member of this blog may post a comment.