Skip to content
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

update url #26

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

update url #26

wants to merge 3 commits into from

Conversation

windy
Copy link

@windy windy commented Aug 26, 2019

新版本的微信公众号授权修改了 BASE URL,我的PR进行了更新。

同时,稍优化了一下 README

@windy
Copy link
Author

windy commented Aug 26, 2019

@NeverMin NeverMin self-requested a review August 29, 2019 12:05
Copy link
Owner

@NeverMin NeverMin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @windy , 感谢你的 PR,可以同步更新一下测试吗?

@sunteya
Copy link

sunteya commented Sep 3, 2019

我们这里也遇到了这个问题,
如果是新的关注用户用之前那个 qrconnect 的地址, 会显示 scope 错误.
我这里已经降回旧版本用 oauth2/authorize 这个地址了.

@Eric-Guo
Copy link
Collaborator

Eric-Guo commented Sep 3, 2019

@sunteya 所以我的理解是这个PR(使用oauth2/authorize)的改动你已经测试过了,可以直接发布,对吗?

@NeverMin
Copy link
Owner

NeverMin commented Sep 5, 2019

场景一 https://open.weixin.qq.com/connect/oauth2/authorize 公众号(服务号)平台

场景二 https://open.weixin.qq.com/connect/qrconnect 微信开放平台

上面的图片分别以场景一场景二作为下文引用说明:
场景一为公众号平台来源,使用的 URL 为/connect/oauth2/authorize针对移动应用,如果在桌面浏览器上打开,会有场景一的提示(请在在微信客户端打开链接);

场景二为是目前 gem 的行为,是微信开放平台来源,使用的 URL 为 /connect/qrconnect,如果使用手机端打开,你需要拿第二台手机或是截屏再让微信扫相册的图片才可以登录。

我个人比较偏向于通过 snsapi_userinfosnsapi_login 去切换,但是否共用一个 URL 这部分我还没有测试(改 callback 等腾讯审核中)。

题外话,如果使用 snsapi_userinfo 的话,腾讯也有来自 Apple 的无奈(大家每年缴600给我,让我也无奈无奈吧)

3、对于iOS应用,考虑到iOS应用商店审核指南中的相关规定,建议开发者接入微信登录时,先检测用户手机是否已安装微信客户端(使用sdk中isWXAppInstalled函数 ),对未安装的用户隐藏微信登录按钮,只提供其他登录方式(比如手机号注册登录、游客登录等)。

@windy
Copy link
Author

windy commented Sep 6, 2019

开放平台的有专门的 gem 包: https://github.com/mycolorway/omniauth-open-wechat-oauth2

所以这边只用在场景一。

@NeverMin
Copy link
Owner

NeverMin commented Sep 6, 2019

看来 README.md 要大改动了。

@windy
Copy link
Author

windy commented Sep 6, 2019

@NeverMin 如果支持两个场景, 那就明确区分开。两个 URL,还可以讲一下如何支持 unionid .

@sunteya
Copy link

sunteya commented Sep 6, 2019

@sunteya 所以我的理解是这个PR(使用oauth2/authorize)的改动你已经测试过了,可以直接发布,对吗?

@Eric-Guo 我这里的做法是把 版本号降回了 0.1.0, 但问题主要是由于 授权地址不对导致的.
公众号的授权需要使用 https://open.weixin.qq.com/connect/oauth2/authorize
而开放平台的授权需要使用 https://open.weixin.qq.com/connect/qrconnect

这个 Pull Request 看着只修改了这些内容, 所以应该是没问题的.

NaiveCAI added a commit to NaiveCAI/omniauth-wechat-oauth2 that referenced this pull request Feb 10, 2021
NaiveCAI added a commit to NaiveCAI/omniauth-wechat-oauth2 that referenced this pull request Feb 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants