Skip to content

update url#26

Open
windy wants to merge 3 commits into
NeverMin:masterfrom
dao42:update_url
Open

update url#26
windy wants to merge 3 commits into
NeverMin:masterfrom
dao42:update_url

Conversation

@windy

@windy windy commented Aug 26, 2019

Copy link
Copy Markdown

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

同时,稍优化了一下 README

@windy

windy commented Aug 26, 2019

Copy link
Copy Markdown
Author

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

@NeverMin NeverMin left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

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

@sunteya

sunteya commented Sep 3, 2019

Copy link
Copy Markdown

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

@Eric-Guo

Eric-Guo commented Sep 3, 2019

Copy link
Copy Markdown
Collaborator

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

@NeverMin

NeverMin commented Sep 5, 2019

Copy link
Copy Markdown
Owner

场景一 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

windy commented Sep 6, 2019

Copy link
Copy Markdown
Author

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

所以这边只用在场景一。

@NeverMin

NeverMin commented Sep 6, 2019

Copy link
Copy Markdown
Owner

看来 README.md 要大改动了。

@windy

windy commented Sep 6, 2019

Copy link
Copy Markdown
Author

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

@sunteya

sunteya commented Sep 6, 2019

Copy link
Copy Markdown

@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