Skip to content

add code.json and generate-codejson.yml workflow #132

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

Open
wants to merge 27 commits into
base: master
Choose a base branch
from

Conversation

leslie-corbalt
Copy link

Description of changes:
add code.json and generate-codejson.yml workflow to comply with ShareIT

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Henry Soule and others added 27 commits February 23, 2023 13:26
This is because we use GITHUB_TOKEN as a task environment
variable. Environment variables are stored in the clear in
a task definition. When the task definition is registered, the token
is available in the task definition.

This is okay because the token expires at the end of a GitHub Actions
job. However, if the task definition is registered first (and the token is exposed) and
the task takes a while to run then a valid token is exposed for the
duration of the long-running task.

This commit runs the task first. Immediately after the task finishes
running, the task definition is registered. Since this action is the
last step of the job, the token expires after the registration completes.
The token exposd in the task definition is invalid.
run the task before registering the task definition
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