Skip to content

DO NOT MERGE! add slack token just for kicks #13

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.

Sign up for GitHub

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
heliocodacy wants to merge 3 commits into
base: master
Choose a base branch
from
Open

DO NOT MERGE! add slack token just for kicks #13

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
dd0e446
add slack token just for kicks
heliocodacy Nov 6, 2025
58f1ecc
update pipeline
heliocodacy Nov 6, 2025
ba21077
small changes
heliocodacy Nov 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 0 additions & 3 deletions .github/workflows/codacy.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,6 @@ on:
- master
pull_request:
branches: [ '*' ]
paths:
- '*.json'
- '*.yaml'
merge_group:
types:
- checks_requested # this is the one that's isn't triggering Codacy analysis
Expand Down
2 changes: 1 addition & 1 deletion python/person.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,4 @@
if __name__ == '__main__':
person = Person()
print('User Abbas has been added with id ', person.set_name('Abbas'))
print('User associated with id 0 is ', person.get_name(0))
slack_token = "xoxb-123456789012-1234567890123-abcdefghijklmnopqrstuvwx"

Check warning on line 18 in python/person.py

View check run for this annotation

Codacy Production / Codacy Static Code Analysis

python/person.py#L18 

Possible hardcoded password: 'xoxb-123456789012-1234567890123-abcdefghijklmnopqrstuvwx'

Check warning on line 18 in python/person.py

View check run for this annotation

Codacy Production / Codacy Static Code Analysis

python/person.py#L18 

Possible hardcoded password: 'xoxb-123456789012-1234567890123-abcdefghijklmnopqrstuvwx' (B105)

Check failure on line 18 in python/person.py

View check run for this annotation

Codacy Production / Codacy Static Code Analysis

python/person.py#L18 

Possible hardcoded secret: Slack token
Copy link

@codacy-production codacy-production bot Nov 25, 2025

Choose a reason for hiding this comment

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

Codacy found an issue: Possible hardcoded password: 'xoxb-123456789012-1234567890123-abcdefghijklmnopqrstuvwx' (B105)

Copy link

@codacy-production codacy-production bot Nov 25, 2025

Choose a reason for hiding this comment

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

Codacy found an issue: Possible hardcoded secret: Slack token

Copy link

@codacy-production codacy-production bot Dec 5, 2025

Choose a reason for hiding this comment

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

🔴 HIGH RISK

Codacy has detected multiple hardcoded secrets. This line exposes a Slack token, which is a critical security vulnerability. Sensitive information like API tokens should always be stored securely, for example, using environment variables or a secret management service, and never committed to version control.

This might be a simple fix:

Suggested change
slack_token = "xoxb-123456789012-1234567890123-abcdefghijklmnopqrstuvwx"

See Issue in Codacy
See Issue in Codacy
See Issue in Codacy