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

πŸ” ✨ Keys leakage & docker image update #8

Merged
merged 2 commits into from
Nov 23, 2023

Conversation

linjan
Copy link
Contributor

@linjan linjan commented Nov 23, 2023

πŸ”§ Minor Mars Update πŸ”§

Greetings,

Please find below the recent changes delivered in this pull request:

  1. πŸ”’ Removed static Spotify secrets and replaced them with environment variables. Transparency of secrets is crucial for maintaining good security practices. βœ…

  2. 🐳 Updated homepage Docker image name, in accordance with the official Github repository. It's always best to keep up with the official documentation to ensure stability, efficiency and security. βœ…

Making us create better, safer and more robust systems. Looking forward to your review and comments.

Best regards,
linjan

@linjan linjan marked this pull request as ready for review November 23, 2023 13:35
@borjapazr
Copy link
Owner

Hello, @linjan πŸ‘‹

Thank you very much for opening this PR, which makes all the sense in the world πŸ™

I'll merge it right away.

Regards!

@borjapazr borjapazr merged commit e58b467 into borjapazr:main Nov 23, 2023
1 check passed
Copy link
Owner

@borjapazr borjapazr Nov 23, 2023

Choose a reason for hiding this comment

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

Thank you for noticing this. Although these secrets are test secrets, it is not correct to set them up this way and share them in the repository.

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.

None yet

2 participants