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

补完 Auth 部分 #4

Merged
merged 2 commits into from
Feb 28, 2018
Merged

补完 Auth 部分 #4

merged 2 commits into from
Feb 28, 2018

Conversation

weizhenye
Copy link
Contributor

一些问题:

  1. Swagger UI 提供了整套 OAuth 2.0 的模拟流程,但在重定向成功后 https://bgm.tv/oauth/access_token 因为 CORS 而 fetch 失败。是否需要在 https://bangumi.github.io/api 提供一个能走通的验证流程?如果 Bangumi 后端根据用户填的回调地址,给 https://bgm.tv/oauth/access_token 的 Header 里动态设置 Access-Control-Allow-Origin,会有安全问题吗?
  2. GET /user/woozy/collections/game 返回以下响应,缺少什么参数吗?
{
  "request": "/user/woozy/collections/game",
  "code": 404,
  "error": "Error: APP ID Parameter is Missing"
}
  1. /collection/{subject_id}/create 默认是 wish 类型,有参数可以传其他类型吗?

@Sai
Copy link
Member

Sai commented Feb 26, 2018

  1. API 的基础约定还没写,需要在所有请求 query 里面加上开发者后台申请的 app_id={YOU_APP_ID}

  2. 这个的字段和 POST /collection/:subject_id/update 是一样的,我更新了文档

@Sai Sai merged commit bfdca87 into bangumi:master Feb 28, 2018
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.

2 participants