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

HAWNG-395: Handle warnings and false-positives #302

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

phantomjinx
Copy link
Member

  • .gitleaks.toml

  • Provides an extension config which lists the false-positives that should be ignored

  • generate-.*.sh

  • Fixes shellcheck warnings is assignments

  • nginx.sh

  • Removes pipefail setting as non-POSIX and since pipes not actually used in script then removed entirely

  • Replaces bash (-v) which POSIX compatible equivalent

  • ws-handler.ts

  • Checks the origin of the MessageEvent. Since the messages are expected from the cluster they should only comes from ${APP_HOST}/master so any other origin is not appropriate so should be ignored

  • .snyk

  • Exclude non-production files from snyk analysis

@phantomjinx phantomjinx self-assigned this Jan 17, 2024
* .gitleaks.toml
 * Provides an extension config which lists the false-positives that
   should be ignored

* generate-.*\.sh
 * Fixes shellcheck warnings is assignments

* nginx.sh
 * Removes pipefail setting as non-POSIX and since pipes not actually used
   in script then removed entirely
 * Replaces bash (-v) which POSIX compatible equivalent

* ws-handler.ts
 * Checks the origin of the MessageEvent. Since the messages are expected
   from the cluster they should only comes from ${APP_HOST}/master so any
   other origin is not appropriate so should be ignored

* .snyk
 * Exclude non-production files from snyk analysis
@phantomjinx phantomjinx merged commit 19e4311 into hawtio:main Jan 17, 2024
3 checks passed
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