AppFlowy-IO / AppFlowy-Cloud

AppFlowy is an open-source alternative to Notion. You are in charge of your data and customizations. Built with Flutter and Rust.
GNU Affero General Public License v3.0
1.05k stars 217 forks source link

[Bug] Unhandled param in url: provider_token #246

Closed rileyhawk1417 closed 8 months ago

rileyhawk1417 commented 10 months ago

Describe the bug In the code provider_token is not handled, which throws an error when logging in via OAuth.

To Reproduce Steps to reproduce the behavior:

  1. Download AppFlowy 0.4.1
  2. Launch and change server to AppFlowyCloud
  3. Click user and log in using any of the OAuth buttons.
  4. After confirming login from the browser check AppFlowy terminal log.

Expected behavior Log into AppFlowy and sync data.

Screenshots If applicable, add screenshots to help explain your problem.

The screenshot below is the error that appears on AppFlowy after trying to sign in using OAuth. AppFlowy Application error

This screenshot are the logs when AppFlowy is run from the terminal. Terminal Logs

Desktop (please complete the following information):

Additional context

speed2exe commented 10 months ago

@rileyhawk1417

Hi thanks for filing the issue. I dont think it's because of provider_token. So in the cloud, there are 2 services that appflowy connects to:

I cant reproduce the error though. Just checking, you've set the server url to https://beta.appflowy.cloud?

speed2exe commented 10 months ago

@rileyhawk1417 I think I might have found the bug, you probably put extra / at the end of the url in cloud settings.

correct:   https://beta.appflowy.cloud
incorrect: https://beta.appflowy.cloud/

This bug will be fixed in the future iteration

rileyhawk1417 commented 10 months ago

Thanks it worked. Should the issue remain open or just closed?

speed2exe commented 10 months ago

@rileyhawk1417 we can keep this open until the issue is fixed.