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.

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

feat: add greetings file #7

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

CBID2
Copy link

@CBID2 CBID2 commented Oct 2, 2023

Description

This PR adds a welcome.yml file.

Type of PR

[X] ✨ Feature
[] 🐞 Bug fix
[] πŸ“„ Documentation update
[] Others

Related Issue

Closes #3

Copy link

@geoffreylgv geoffreylgv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @CBID2, this greeting GitHub action is nice tho.
I've been analyzing it since I saw it from Eddiehub repo.
Personally, I use the actions/first-interaction@v1, have you tried it before?

@CBID2
Copy link
Author

CBID2 commented Oct 3, 2023

Hi @CBID2, this greeting GitHub action is nice tho.

I've been analyzing it since I saw it from Eddiehub repo.

Personally, I use the actions/first-interaction@v1, have you tried it before?

No tell me more @geoffreylgv. :)

@geoffreylgv
Copy link

Hi @CBID2, this greeting GitHub action is nice tho.
I've been analyzing it since I saw it from Eddiehub repo.
Personally, I use the actions/first-interaction@v1, have you tried it before?

No tell me more @geoffreylgv. :)

Alright, It's also simple to use and pr or issue message customization; similar to the one you added

@geoffreylgv
Copy link

Allow me to suggest something, on the call of the github.actor, if you put the @ it will notice he/her (the actor)

 @${{ github.actor }}

Copy link
Collaborator

@saragerion saragerion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @CBID2, thanks so much for opening this PR and contributing to shesharp! :)

This PR looks good, but I see an opportunity to harden the security of this Github workflow.

  1. The GITHUB_TOKEN secret used in this action currently has write permissions on all scopes. This is more permissions than is necessary to perform its task in this action.
    Can you reduce the scope of the token permission to write on only the relevant scopes (issues and pull-requests).
    Reference: https://docs.github.com/en/actions/using-jobs/assigning-permissions-to-jobs#defining-access-for-the-github_token-scopes

See also:
Use credentials that are minimally scoped

  1. I am ok with leveraging a community-created action but I would prefer if we pin the version by using a commit SHA rather than a branch or a tag. Using a SHA is generally suggested because of various reasons:
  • It makes the third-party action's code immutable: the source action won't change over time with new releases so the action will always have a predictable behavior. New breaking changes won't impact our workflow.
  • It helps mitigate the risk of a bad actor adding a backdoor to the third-party action's repository. Pinning to a commit SHA reduces this risk of unintentionally introducing malicious code in the future ( a branch or tag of an action can be compromised, the commit SHA is immutable).

See more info here: https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions#using-third-party-actions


Related: while looking at this PR I realized that we can improve the security posture of other shesharp repositories as well so I created these new issues to address that effort:

shesharpnl/hackathon-2023.slack-bot.base-example#1
shesharpnl/hackathon-2021.twitter-bot.base-example#8
shesharpnl/hackathon-2021.slack-bot.base-example#1

@CBID2
Copy link
Author

CBID2 commented Oct 6, 2023

Hey @saragerion. I made the necessary changes

runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: EddieHubCommunity/gh-action-community/src/welcome@bf67289baedda0b13e216f7ca14e733eab899963
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @CBID2, thanks for implementing this change.

Question: where did you find this commit SHA?
I do not see it in the main branch:
https://github.com/EddieHubCommunity/gh-action-community/commits/main
And this URL doesn't work:
https://github.com/EddieHubCommunity/gh-action-community/tree/bf67289baedda0b13e216f7ca14e733eab899963

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adiati98
Copy link
Collaborator

Hey @CBID2,
How's the progress here? Do you need more time or help?

@CBID2
Copy link
Author

CBID2 commented Jan 18, 2024

Hey @CBID2,

How's the progress here? Do you need more time or help?

Hey @adiati98. I tried finding the latest version of the greeting in EddieHub's repo to no avail. It's needed in order to solve the security part.

@adiati98
Copy link
Collaborator

Hey @CBID2,
How's the progress here? Do you need more time or help?

Hey @adiati98. I tried finding the latest version of the greeting in EddieHub's repo to no avail. It's needed in order to solve the security part.

Do you want to keep working on this?

@CBID2
Copy link
Author

CBID2 commented Jan 18, 2024

Hey @CBID2,

How's the progress here? Do you need more time or help?

Hey @adiati98. I tried finding the latest version of the greeting in EddieHub's repo to no avail. It's needed in order to solve the security part.

Do you want to keep working on this?

Yes

@adiati98
Copy link
Collaborator

Hey @CBID2,

How's the progress here? Do you need more time or help?

Hey @adiati98. I tried finding the latest version of the greeting in EddieHub's repo to no avail. It's needed in order to solve the security part.

Do you want to keep working on this?

Yes

Awesome! Please keep us posted. πŸ‘

And do let us know if you can't go further. We can close this and you can always create a new issue and PR once you figure something out. πŸ™‚

@adiati98
Copy link
Collaborator

adiati98 commented Feb 7, 2024

@CBID2 I want to inform that as there is no progress for this PR in the last 3 weeks, I'm putting it on draft. Feel free to change the status as "ready for review" once you've resolved it.

However, because it's already open for 4 months, if you haven't resolved it by next week (February 14th), I will close this PR. You can open a new issue and PR in that case if you're still interested. Thank you. 😊

cc: @saragerion

@adiati98 adiati98 marked this pull request as draft February 7, 2024 08:58
@CBID2 CBID2 marked this pull request as ready for review February 7, 2024 22:37
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.

Add a greeting.yml file
4 participants