Skip to content

chore: Temporary disable tag alert actions #100

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

Merged
merged 1 commit into from
Oct 6, 2020
Merged

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.

2 participants