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

Add a new DB error type for validation errors #189

Merged
merged 5 commits into from
Mar 26, 2023
Merged

Conversation

FrimIdan
Copy link
Member

@FrimIdan FrimIdan commented Mar 26, 2023

Description

The PR adds a new DB error (bad request) for object validation errors for the following:

  • finding
  • scan
  • scan_config
  • scan_result
  • target

It also adds 400 response for the relevant APIs in the openapi.yaml.

Type of Change

[ ] Bug Fix
[X] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@FrimIdan FrimIdan requested a review from a team as a code owner March 26, 2023 12:19
@FrimIdan FrimIdan force-pushed the bad-req-response branch 2 times, most recently from 640d69a to 9c92ac7 Compare March 26, 2023 14:42
@FrimIdan FrimIdan merged commit 51eee8f into main Mar 26, 2023
@FrimIdan FrimIdan deleted the bad-req-response branch March 26, 2023 15:22
shalevr added a commit that referenced this pull request Mar 28, 2023
* main: (31 commits)
  Improve error messages and add missing comments in Lynis scanner
  Add lynis scanner to the misconfiguration family
  fix nil summary crash in Region findings widget (#194)
  Switch result processor vulnerabilties map print to debug level (#192)
  UI dashboard (#191)
  set disabled to true for a non periodic scan once scan started
  Add infrastructure for scan result post processing (#136)
  Add a new DB error type for validation errors (#189)
  Findings trends dashboard widget (#182)
  Fix odata queries for TargetScanResult/Status (#184)
  UI null fields (#188)
  Fix exporter check in waitFroAttached
  Malware family and ClamAV implementation (#118)
  Propagate Context (#181)
  Add support for comparing datetimes to odatasql
  Most risky assets dashboard widget (#172)
  Make sure that scan init failures are captured by scan state (#144)
  UI config updates new (#180)
  Fix error message in misconfiguration family
  Add missing misconfiguration scan scanners field
  ...

# Conflicts:
#	cli/cmd/exportresults.go
#	cli/cmd/root.go
#	runtime_scan/pkg/config/config.go
#	runtime_scan/pkg/scanner/job_managment.go
shalevr added a commit that referenced this pull request Mar 28, 2023
* main: (31 commits)
  Improve error messages and add missing comments in Lynis scanner
  Add lynis scanner to the misconfiguration family
  fix nil summary crash in Region findings widget (#194)
  Switch result processor vulnerabilties map print to debug level (#192)
  UI dashboard (#191)
  set disabled to true for a non periodic scan once scan started
  Add infrastructure for scan result post processing (#136)
  Add a new DB error type for validation errors (#189)
  Findings trends dashboard widget (#182)
  Fix odata queries for TargetScanResult/Status (#184)
  UI null fields (#188)
  Fix exporter check in waitFroAttached
  Malware family and ClamAV implementation (#118)
  Propagate Context (#181)
  Add support for comparing datetimes to odatasql
  Most risky assets dashboard widget (#172)
  Make sure that scan init failures are captured by scan state (#144)
  UI config updates new (#180)
  Fix error message in misconfiguration family
  Add missing misconfiguration scan scanners field
  ...

# Conflicts:
#	cli/cmd/exportresults.go
#	cli/cmd/root.go
#	runtime_scan/pkg/config/config.go
#	runtime_scan/pkg/scanner/job_managment.go
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