Skip to content

fix: secure greetings workflow by hardening permissions#342

Open
zarimaaz55-collab wants to merge 1 commit into
ajay-dhangar:mainfrom
zarimaaz55-collab:patch-1
Open

fix: secure greetings workflow by hardening permissions#342
zarimaaz55-collab wants to merge 1 commit into
ajay-dhangar:mainfrom
zarimaaz55-collab:patch-1

Conversation

@zarimaaz55-collab

Copy link
Copy Markdown

This PR fixes the security vulnerability in the greetings workflow by hardening the permissions block. It sets a default top-level read permission for contents and restricts write permissions specifically to the greeting job. Emojis have also been cleaned up.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for opening a pull request in CodeHarborHub! 🎉 We appreciate your contribution and will review it shortly.

@sonarqubecloud

Copy link
Copy Markdown

@deepsource-io

deepsource-io Bot commented Jun 22, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 8a23ff1...22e5a43 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Jun 22, 2026 5:42p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@ajay-dhangar ajay-dhangar left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

update your code based on suggestions

with:
repo-token: ${{ secrets.GITHUB_TOKEN }}
issue-message: >
Thanks for opening an issue in CodeHarborHub!

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

replace CodeHarborHub! with my Portfolio or maybe our repo

Thanks for opening an issue in CodeHarborHub!
We'll get back to you as soon as possible.
pr-message: >
Thanks for opening a pull request in CodeHarborHub!

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

replace CodeHarborHub! with my Portfolio or maybe our repo

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