Skip to content

Add gh action to upload artifacts#398

Merged
G10h4ck merged 8 commits into
libremesh:developfrom
selankon:f/upload-artifacts
Feb 12, 2024
Merged

Add gh action to upload artifacts#398
G10h4ck merged 8 commits into
libremesh:developfrom
selankon:f/upload-artifacts

Conversation

@selankon
Copy link
Copy Markdown
Collaborator

@selankon selankon commented Feb 9, 2024

FTR

Limeapp automatic preview builds.

Now when a PR is marked with the Labels "artifact", an additional GH action will run to upload a built version of the limeapp on GH.

This artifact will store the PR number, to on posterior publish a GH comment on the PR with the artifacts url.

Example:

#398 (comment)

The github action to publish the comment runs on the develop branch and not on the pr to protect from external forks PR's the secrets and other configurations (to comment on the PR the action needs write and read permissions, which are dangerous to give to a fork from an external repository).

Further reading

Keeping your GitHub Actions and workflows secure

https://securitylab.github.com/research/github-actions-preventing-pwn-requests/

https://glasnt.com/blog/pull_request_target_labels/

Workflow run docs

https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_run
https://docs.github.com/en/webhooks/webhook-events-and-payloads#workflow_run

@selankon selankon added the artifact Will triger a GH action to create an artifact label Feb 9, 2024
@selankon selankon force-pushed the f/upload-artifacts branch 2 times, most recently from 94e7553 to bcaa121 Compare February 9, 2024 17:50
@selankon selankon added artifact Will triger a GH action to create an artifact and removed artifact Will triger a GH action to create an artifact labels Feb 9, 2024
@selankon selankon added artifact Will triger a GH action to create an artifact and removed artifact Will triger a GH action to create an artifact labels Feb 12, 2024
@selankon selankon added artifact Will triger a GH action to create an artifact and removed artifact Will triger a GH action to create an artifact labels Feb 12, 2024
@selankon selankon force-pushed the develop branch 12 times, most recently from e11390e to c749df1 Compare February 12, 2024 17:42
@github-actions
Copy link
Copy Markdown

@G10h4ck G10h4ck merged commit b292539 into libremesh:develop Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

artifact Will triger a GH action to create an artifact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants