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.
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
In my case I was returing with empty return:
if (...)
return; // problem here
To fix, I returned the observed object:
if (...)
return req.next();
For me, I had a subscription piped with takeUntil
like this:
this.route.queryParams
.pipe(takeUntil(this.ngUnsubscribe))
.subscribe(async(queryParams) => {
});
I forgot to initialize this.ngUnsubscribe
with an observable. When I set the value to an observable in the component (ngUnsubscribe = new Subject()
), the error was gone.
I did face this issue and in my case responseType
suppose to be text
(because of designed API end-point) rather than default json
.
import { HttpClient } from '@angular/common/http';
Before:
getToken(tokenCommand) {
return this.http.post(API_BASE_URL + 'user/token', tokenCommand);
}
After fixed:
getToken(tokenCommand) {
return this.http.post(API_BASE_URL + 'user/token', tokenCommand
, { responseType: 'text' });
}
I think this error message is too general and it will be nice if it's developers could provide more detail/helpful error message. Thanks.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With