Skip to content

Conversation

@potato4d
Copy link
Member

@potato4d potato4d commented Oct 5, 2020

Description of Problem

  • secrets.GITHUB_TOKEN は固定化されたトークンであり、その PR 発行者の GitHub のトークンを利用する
    • そのため権限のないユーザーからの PR については、適切にラベルを付与する処理ができない

Proposed Solution

  • 暫定的対処として、現状存在が特に必要ない処理を STOP する
    • JA_GITHUB_TOKEN を用意し、 @potato4d の個人アカウントを紐付ける事も考えたが、アクセスコントロールを狭めることができないためセキュリティ上難しい

Additional Information

  • @vuejs-jp-bot でトークンを発行する手段はあるが、その場合該当 Bot がそのトークンにてアクセスできる範囲を最小化することが必要となる(public repo へのアクセスくらいの粒度しかできないので、難しそう)

@netlify
Copy link

netlify bot commented Oct 5, 2020

Deploy preview for vuejs-v3-ja-doc-preview ready!

Built with commit 2e8a9c4

https://deploy-preview-100--vuejs-v3-ja-doc-preview.netlify.app

@kazupon
Copy link
Member

kazupon commented Oct 6, 2020

Github の token 周りのコントロール、柔軟に作られていないくて、微妙ですよね...

@kazupon kazupon merged commit f900eea into lang-ja Oct 6, 2020
@kazupon kazupon deleted the fix/github-token branch October 6, 2020 01:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants