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

Use of Hard-coded Password #357

Closed
rajeevdodda opened this issue Aug 24, 2021 · 6 comments
Closed

Use of Hard-coded Password #357

rajeevdodda opened this issue Aug 24, 2021 · 6 comments

Comments

@rajeevdodda
Copy link

When we scanned our application using a static scan tool, it reported a security vulnerabity in the below line with a category of "Use of Hard-coded Password"

var ReactPropTypesSecret = 'SECRET_DO_NOT_PASS_THIS_OR_YOU_WILL_BE_FIRED';

@sdoppalapudi
Copy link

When we scanned our application using a static scan tool(Veracode), it reported a security vulnerabity in the below line with a category of "Use of Hard-coded Password"

var ReactPropTypesSecret = 'SECRET_DO_NOT_PASS_THIS_OR_YOU_WILL_BE_FIRED';

@ljharb
Copy link
Collaborator

ljharb commented Aug 24, 2021

Then your tool is pretty silly - just because a variable has the name “secret” doesn’t mean it’s actually a secret value, and your tool shouldn’t be warning you about third-party code.

@ljharb ljharb closed this as completed Aug 24, 2021
@sdoppalapudi
Copy link

mean

@ljharb Thanks for the reply. But the scan tool veracode(static code scan) will scan all the build files and libraries used in the application. So, it reported this issue.

Looks it found a string 'SECRET_DO_NOT_PASS_THIS_OR_YOU_WILL_BE_FIRED' which has some SECRET in it. So, as part of our process, we need to report this vulnerability to the library GitHub and take confirmation.

can you please confirm is this an issue? I mean this piece of code in the library will not create any issue?.

@ljharb
Copy link
Collaborator

ljharb commented Aug 26, 2021

It is not an issue, and it is not a secret. The word “secret” isn’t a vulnerability.

@sdoppalapudi
Copy link

Thanks for the confirmation @ljharb.

@ljharb
Copy link
Collaborator

ljharb commented Aug 26, 2021

There’s no point in scanning third-party code for secrets, even if it’s included in your application, because they won’t be your secrets, and because there’s nothing you can do to fix it anyways.

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

3 participants