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

GitHub secret scanner detects 'google-custom-search-api-key' => 'AIzaSyDpE01VDNNT73s6CEeJRdSg5jukoG244ek' as a problem secret #6115

Open
rfay opened this issue Sep 23, 2024 · 4 comments

Comments

@rfay
Copy link

rfay commented Sep 23, 2024

Describe the bug

GitHub's secret scanner detects this as a secret:

public function fetch($search = null, $options = ['image-viewer' => 'open', 'google-custom-search-api-key' => 'AIzaSyDpE01VDNNT73s6CEeJRdSg5jukoG244ek']
 

#[CLI\Option(name: 'google-custom-search-api-key', description: 'Google Custom Search API Key, available from https://code.google.com/apis/console/. Default key limited to 100 queries/day globally.')]

It's probably private, but the scan alert is in

@weitzman
Copy link
Member

That's ok. That code is not actually secret. If these alerts bother folks I suggest only scanning your customer code and not scanning dependencies.

@rfay
Copy link
Author

rfay commented Sep 23, 2024

This isn't a manual scan, it's an automatic GitHub check on checkin of code. And yes, I check in vendor because I'm lazy.

I'm surprised they haven't been pestering you over and over.

@weitzman
Copy link
Member

weitzman commented Oct 4, 2024

Is there a code comment which will disable scanning for this file or line?

@rfay
Copy link
Author

rfay commented Oct 4, 2024

I guess this is something we have enabled, and it's a good idea

IMO just changing the content of the key to your-key-here might prevent the notification.

It's a great feature. I don't see a way to disable it for one line.

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

No branches or pull requests

2 participants