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.

Sign up for GitHub

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

Jump to bottom

需要支持 Notebook 和其他服务之间的互信 token 设置 #159

Closed
chncaesar opened this issue Jun 7, 2022 · 1 comment
Closed

需要支持 Notebook 和其他服务之间的互信 token 设置 #159

chncaesar opened this issue Jun 7, 2022 · 1 comment
Assignees
@allwefantasy
Labels
byzer-notebook 1.2.1
Milestone
Sprint-07/01

Comments

@chncaesar
Copy link
Contributor

chncaesar commented Jun 7, 2022 •
edited by ZhengshuaiPENG
Loading

当前情况 token 是通过 jwt 代码 NotebookApiService.getNotebook 代码中使用写死私钥 parse jwt token。这导致第三方调用 notebook Open APi时，使用同样 jwt token，且不可更改。

需要让用户通过参数进行配置这个 token，来保证不同 notebook 在实现跨应用互信时的 token 不同
@ZhengshuaiPENG ZhengshuaiPENG changed the title 支持动态配置 jwt 加解密私钥 需要支持 Notebook 和其他服务之间的互信 token 设置 Jun 29, 2022
@ZhengshuaiPENG ZhengshuaiPENG added this to the Sprint-07/01 milestone Jun 29, 2022
@ZhengshuaiPENG ZhengshuaiPENG removed their assignment Jun 29, 2022
ZhengshuaiPENG pushed a commit that referenced this issue Jun 29, 2022
@allwefantasy
#159 Add notebook.services.communication.token to allow the third-par…
06d8f9b 
…ty service accessing notebook (#163)

* open api secret key is the same with notebook.security.key

* add notebook.service.security.key to allow the third party accessing notebook

* rename notebook.services.security.key to  notebook.services.communication.token
@ZhengshuaiPENG
Copy link
Contributor

新增参数配置 notebook.services.communication.token, 该配置是可以通过在其他应用中需要和 notebook 联动时，配置相同的 token，来完成服务端的通信认证

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@allwefantasy allwefantasy

Labels
byzer-notebook 1.2.1
Projects
None yet
Milestone
Sprint-07/01
Development

No branches or pull requests
3 participants
@allwefantasy @ZhengshuaiPENG @chncaesar