Skip to content

[CDTOOL-1295] Add support for API Discovery and API Inventory Operations #1689

Merged
rcaril merged 12 commits intomainfrom
rcaril/CDTOOL-1295-add-support-for-security-operations
Mar 16, 2026
Merged

[CDTOOL-1295] Add support for API Discovery and API Inventory Operations #1689
rcaril merged 12 commits intomainfrom
rcaril/CDTOOL-1295-add-support-for-security-operations

Conversation

@rcaril
Copy link
Contributor

@rcaril rcaril commented Mar 13, 2026 •
edited
Loading

Change summary

This PR adds CRUD support for the api-security operations and api-security discovered-operations commands.

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

  • Does your submission pass tests?

Changes to Core Features:

  • Have you written new tests for your core changes, as applicable?
  • Have you successfully run tests with your changes locally?

@rcaril rcaril marked this pull request as ready for review March 13, 2026 19:35
@rcaril rcaril requested a review from a team as a code owner March 13, 2026 19:35
rcaril
rcaril commented Mar 13, 2026
View reviewed changes
rcaril added 2 commits March 13, 2026 15:40
@rcaril
Fix commands.go syntax
7359592
@rcaril
fix tests
7e7619c 
removed meta from checks and updated run_test.go
@rcaril rcaril changed the title [CDTOOL-1295] Add support for API Security Operations ('discovered-operations' and 'operations) [CDTOOL-1295] Add support for API Security and API Inventory Mar 13, 2026
@kpfleming kpfleming changed the title [CDTOOL-1295] Add support for API Security and API Inventory [CDTOOL-1295] Add support for API Discovery and API Inventory Operations Mar 16, 2026
kpfleming
kpfleming approved these changes Mar 16, 2026
View reviewed changes
Copy link
Contributor

@kpfleming kpfleming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One tiny doc change, otherwise this looks good to me. I really don't understand why the API doesn't offer GET for a discovered-operation, but that's a topic for a different channel.

@rcaril rcaril merged commit 599bf05 into main Mar 16, 2026
13 checks passed
@rcaril rcaril deleted the rcaril/CDTOOL-1295-add-support-for-security-operations branch March 16, 2026 19:22
@BrewTestBot BrewTestBot mentioned this pull request Mar 17, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anthony-gomez-fastly anthony-gomez-fastly anthony-gomez-fastly left review comments

@kpfleming kpfleming kpfleming approved these changes

@philippschulte philippschulte Awaiting requested review from philippschulte philippschulte is a code owner automatically assigned from fastly/developer-tools

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rcaril @kpfleming @anthony-gomez-fastly