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.

Sign up for GitHub

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

Jump to bottom

feat: bed-5132 - updated openapi spec for posture page api contract #1047

Open
wants to merge 6 commits into
base: BED-5083
Choose a base branch
from
Open

feat: bed-5132 - updated openapi spec for posture page api contract #1047

wants to merge 6 commits into from
+361 −40

Conversation

sircodemane
Copy link
Contributor

Description

  • updated finding trends endpoint openapi spec
  • updated posture history endpoint openapi spec

Motivation and Context

This PR addresses: BED-5132

Modifies the OpenAPI spec for posture page endpoints in preparation for aggregated data updates

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

@sircodemane sircodemane added documentation Improvements or additions to documentation enhancement New feature or request api A pull request containing changes affecting the API code. labels Jan 3, 2025
@sircodemane sircodemane self-assigned this Jan 3, 2025
@elikmiller elikmiller changed the base branch from main to BED-5083 January 10, 2025 18:53
@sircodemane
feat: BED-5132 - api contract update
febbc03 
- fixed incorrect posture history data type enum values
- added fields to finding-trends for impact and exposure values
@sircodemane
chore: bed-5132: api contract chores
f7afef8 
- added missing attack-paths posture history data type
- regenerated openapi.json
- captured some missed cue file generation
@sircodemane
feat: bed-5132 api contract
de60c3d 
- added `environment_ids` field to finding-trends response
@sircodemane
Merge branch 'BED-5083' into bed-5132-api-contract
88f2a95
@sircodemane
fix: BED-5132 - added composite risk field because it turns out we ac…
4272426 
…tually needed it
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benwaples benwaples benwaples left review comments

@elikmiller elikmiller elikmiller left review comments

Assignees

@sircodemane sircodemane

Labels
api A pull request containing changes affecting the API code. documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sircodemane @elikmiller @benwaples