This PR is the core for a new minor release which adds REST API endpoints for all operations executed from the frontend plugin that currently interact with the PagerDuty REST API directly. With this change we:
Remove the dependency on the Backstage proxy
Improve security by limiting the actions performed on the backend API
Slightly increase the performance by limiting the data used by the frontend to the essential
Endpoints added
/oncall-users - returns PagerDutyOnCallUsersResponse with list of users oncall
/services - uses integration_key and returns PagerDutyServiceResponse with PagerDuty service information
/services/:serviceId - returns PagerDutyServiceResponse with PagerDuty service information
/services/:serviceId/change-events - returns PagerDutyChangeEventsResponse with list of last 5 change events for the defined service
/services/:serviceId/incidents - returns PagerDutyIncidentsResponse with list of incidents for the defined service
With this change, the proxy configuration on app-config.yaml is no longer required.
Acknowledgement
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.
Description
This PR is the core for a new minor release which adds REST API endpoints for all operations executed from the frontend plugin that currently interact with the PagerDuty REST API directly. With this change we:
Endpoints added
/oncall-users - returns PagerDutyOnCallUsersResponse with list of users oncall /services - uses integration_key and returns PagerDutyServiceResponse with PagerDuty service information /services/:serviceId - returns PagerDutyServiceResponse with PagerDuty service information /services/:serviceId/change-events - returns PagerDutyChangeEventsResponse with list of last 5 change events for the defined service /services/:serviceId/incidents - returns PagerDutyIncidentsResponse with list of incidents for the defined service
With this change, the proxy configuration on
app-config.yamlis no longer required.Acknowledgement
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.