Feature | Extend Swagger Coverage for controller OAuth2TagsApiController

[matiasperrone-exo]

Task:

Ref: https://app.clickup.com/t/86b6wkh6w

[caseylocker]

@matiasperrone-exo The security schema needs to be updated/created with the correct scopes.

Also, your missing Security on GET /api/v1/tags/{id}
This endpoint has NO security parameter, but the seeder shows it requires authentication:

operationIds needed

Endpoint	Suggested operationId
GET /api/v1/tags	getAllTags
GET /api/v1/tags/{id}	getTag
POST /api/v1/tags	createTag
PUT /api/v1/tags/{id}	updateTag
DELETE /api/v1/tags/{id}	deleteTag

Missing x: ['required-groups'] for Write Operations
POST, PUT, DELETE endpoints require authorization groups per the seeder:
x: [
'required-groups' => [
IGroup::SuperAdmins,
IGroup::Administrators,
IGroup::SummitAdministrators,
]
],

[caseylocker]

Wrong x: Parameter Key
All 3 write endpoints use 'authz_groups' instead of the project standard 'required-groups'. Please change these to 'required-groups'

Affected locations: createTag, updateTag, deleteTag

[matiasperrone-exo]

Thanks @caseylocker for the comments. Now is ready to review again

[caseylocker]

Approved

[smarcet]

@matiasperrone please rebase against main and review conflicts

[smarcet]

@matiasperrone please rebase against main and review conflicts

[matiasperrone-exo]

@smarcet the branch is now rebased and conflicts free.

[smarcet]

LGTM