Skip to content

createAuthUri problem on login #1548

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
okandeb opened this issue Feb 19, 2019 · 8 comments
Closed

createAuthUri problem on login #1548

okandeb opened this issue Feb 19, 2019 · 8 comments

Comments

@okandeb
Copy link

okandeb commented Feb 19, 2019

hello all,
when i try to login get

INVALID_CONTINUE_URI : Invalid parameter value for redirect_uri: fragment not allowed.

with 400 error code on ios. this is the request payload

{"identifier":"*****@gmail.com","continueUri":"http://localhost:8080/Users/okanseremet/Library/Developer/CoreSimulator/Devices/0976F085-0569-488D-8440-403F52BD13C5/data/Containers/Bundle/Application/A96557FB-A081-47E6-AFB4-BC8A097D14B4/MaxiLoyal.app/www/index.html#/welcome"}

but it should be like this. Otherwise it's not working.

{"identifier":"*****@gmail.com","continueUri":"http://localhost:8080"}

i tried to downgrade node version from 10.1 to 8.11.2 but not working
cordova version 7.1.0
ionic version 3.20.0

"dependencies": {
    "@agm/core": "1.0.0-beta.2",
    "@angular/cli": "1.7.0",
    "@angular/common": "5.2.9",
    "@angular/compiler": "5.2.9",
    "@angular/core": "5.2.9",
    "@angular/forms": "5.2.9",
    "@angular/http": "5.2.9",
    "@angular/platform-browser": "5.2.9",
    "@angular/platform-browser-dynamic": "5.2.9",
    "@ionic-native/app-minimize": "4.6.0",
    "@ionic-native/app-version": "4.6.0",
    "@ionic-native/core": "4.17.0",
    "@ionic-native/device": "4.6.0",
    "@ionic-native/diagnostic": "4.6.0",
    "@ionic-native/facebook": "4.6.0",
    "@ionic-native/firebase": "4.11.0",
    "@ionic-native/geolocation": "4.15.0",
    "@ionic-native/google-plus": "^4.11.0",
    "@ionic-native/in-app-browser": "4.6.0",
    "@ionic-native/keyboard": "4.7.0",
    "@ionic-native/local-notifications": "4.9.1",
    "@ionic-native/location-accuracy": "4.16.0",
    "@ionic-native/network": "4.17.0",
    "@ionic-native/nfc": "4.12.2",
    "@ionic-native/qr-scanner": "4.16.0",
    "@ionic-native/social-sharing": "4.6.0",
    "@ionic-native/splash-screen": "4.6.0",
    "@ionic-native/status-bar": "4.6.0",
    "@ionic/cli-plugin-cordova": "1.6.2",
    "@ionic/pro": "1.0.20",
    "@ionic/storage": "2.1.3",
    "@types/google-maps": "3.2.0",
    "angular2-qrcode": "2.0.1",
    "angularfire2": "5.0.0-rc.6",
    "cordova-android": "7.1.0",
    "cordova-android-support-gradle-release": "1.4.4",
    "cordova-ios": "4.5.4",
    "cordova-plugin-add-swift-support": "1.7.2",
    "cordova-plugin-app-event": "1.2.1",
    "cordova-plugin-app-version": "0.1.9",
    "cordova-plugin-appminimize": "1.0.0",
    "cordova-plugin-compat": "1.2.0",
    "cordova-plugin-console": "1.1.0",
    "cordova-plugin-device": "2.0.2",
    "cordova-plugin-facebook4": "1.7.4",
    "cordova-plugin-firebase": "1.0.5",
    "cordova-plugin-geolocation": "git+https://github.com/apache/cordova-plugin-geolocation.git",
    "cordova-plugin-googleplus": "5.3.0",
    "cordova-plugin-inappbrowser": "2.0.2",
    "cordova-plugin-ionic": "4.1.7",
    "cordova-plugin-ionic-webview": "1.1.19",
    "cordova-plugin-network-information": "2.0.1",
    "cordova-plugin-qrscanner": "2.6.0",
    "cordova-plugin-request-location-accuracy": "2.2.3",
    "cordova-plugin-splashscreen": "5.0.2",
    "cordova-plugin-statusbar": "2.4.2",
    "cordova-plugin-swift-support": "3.1.1",
    "cordova-plugin-whitelist": "1.3.3",
    "cordova-plugin-x-socialsharing": "5.3.2",
    "cordova-support-android-plugin": "1.0.1",
    "cordova.plugins.diagnostic": "4.0.5",
    "de.appplant.cordova.plugin.local-notification": "0.8.5",
    "es6-promise-plugin": "4.2.2",
    "firebase": "4.12.1",
    "font-awesome": "4.7.0",
    "ion": "1.0.1",
    "ionic-angular": "3.9.2",
    "ionic-plugin-keyboard": "2.2.1",
    "ionicons": "3.0.0",
    "moment": "2.22.2",
    "ng2-translate": "5.0.0",
    "node-js-marker-clusterer": "1.0.0",
    "phonegap-nfc": "1.0.3",
    "platform": "1.3.5",
    "reflect-metadata": "0.1.12",
    "rxjs": "5.5.10",
    "sw-toolbox": "3.6.0",
    "typescript": "2.6.2",
    "zone.js": "0.8.24"
  },
  "devDependencies": {
    "@angular/compiler-cli": "5.2.9",
    "@ionic/app-scripts": "3.2.0",
    "@types/node": "9.6.1",
    "autoprefixer": "8.2.0",
    "tslint": "5.9.1",
    "tslint-eslint-rules": "5.1.0"
  },

how can i fix this issue? I tried version changing, platform removing and adding etc.

@google-oss-bot
Copy link
Contributor

I found a few problems with this issue:

  • I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.
  • This issue does not seem to follow the issue template. Make sure you provide all the required information.

@bojeil-google
Copy link
Contributor

Are you calling this with fetchProvidersForEmail or fetchSignInMethodsForEmail?

@okandeb
Copy link
Author

okandeb commented Feb 20, 2019

Yes. using fetchProvidersForEmail method. Shouldn't I?

Edit: You are right. When i remove fetchProvidersForEmail method sign in flow works properly. But i have to use the method. Is there any way to check providers for email without using this method or any workaround?
Edit 2: I also tried fetchSignInMethodsForEmail but didn't work.

@bojeil-google
Copy link
Contributor

I think it may be a bug in our backend code. This endpoint is also used for OAuth flows and redirect URIs are not supposed to have fragments per spec. It shouldn't be an issue for fetchProvidersForEmail or fetchSignInMethodsForEmail. Our team is investigating a fix.

@bojeil-google
Copy link
Contributor

You can also file a support ticket via official channels to track the progress.

@wti806
Copy link
Contributor

wti806 commented Feb 26, 2019

@okandeb The issue should be fixed now. Can you try it out and verify? Thanks

@okandeb
Copy link
Author

okandeb commented Feb 27, 2019

@wti806 i will let you know when i try it out. Thanks.

merlinnot pushed a commit to merlinnot/firebase-js-sdk that referenced this issue Mar 11, 2019
@okandeb
Copy link
Author

okandeb commented Mar 15, 2019

problem seems to be solved. Cheers!

@okandeb okandeb closed this as completed Mar 15, 2019
@firebase firebase locked and limited conversation to collaborators Oct 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants