New release 0.8.1 breaks use of different PagerDuty API host
#74
Assignees
Labels
Comments
pcgqueiroz
changed the title
0.8.1 breaks use of different PagerDuty api host0.8.1 breaks use of different PagerDuty API host
|
Thanks for reporting this @pcgqueiroz. While the plugin continues to work it introduces additional latency if your PagerDuty instance is running in Europe for sure. I'll look into this with high priority and add support for an override like we already have for the events url. |
This was referenced Feb 22, 2024
t1agob
added a commit
that referenced
this issue
Feb 22, 2024
### Description This PR resolves an issue reported in backstage-plugin (#74) which prevents users from overriding the REST API base url (e.g. for EU based accounts). This feature was possible through the Backstage proxy configuration. With this, users will be able to add a new configuration to the PagerDuty plugin in `app-config.yaml` like the example below. ```yaml pagerDuty: apiBaseUrl: https://api.eu.pagerduty.com #defaults to https://api.pagerduty.com ``` This feature in mainly implemented in `@pagerduty/backstage-plugin-backend` but this PR updates the plugin config schema accordingly. **Issue number:** #74 ### Type of change - [ ] New feature (non-breaking change which adds functionality) - [x] Fix (non-breaking change which fixes an issue) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) ### Checklist - [x] I have performed a self-review of this change - [x] Changes have been tested - [x] Changes are documented - [x] Changes generate *no new warnings* - [x] PR title follows [conventional commit semantics](https://www.conventionalcommits.org/en/v1.0.0/) If this is a breaking change 👇 - [ ] I have documented the migration process - [ ] I have implemented necessary warnings (if it can live side by side) ## 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.
Merged
t1agob
added a commit
that referenced
this issue
Feb 22, 2024
### Description This release resolves an issue reported in backstage-plugin (#74) which prevents users from overriding the REST API base url (e.g. for EU based accounts). This feature was possible through the Backstage proxy configuration. With this, users will be able to add a new configuration to the PagerDuty plugin in `app-config.yaml` like the example below. ```yaml pagerDuty: apiBaseUrl: https://api.eu.pagerduty.com #defaults to https://api.pagerduty.com ``` This feature in mainly implemented in `@pagerduty/backstage-plugin-backend` but this release updates the plugin config schema accordingly. ## 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.
|
@pcgqueiroz This should be fixed now. Please upgrade to the latest version of the packages and follow the docs on how to configure/override the base REST API url. latest versions: |
github-project-automation
bot
moved this from Under review
to Done
in PagerDuty plugin for Backstage
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
The new release
0.8.1that deprecates the proxy configuration in favor of using a backend plugin to make calls to the PagerDuty REST API does not offer the possibility to change the server base URL. Calls to the european server https://api.eu.pagerduty.com are not possible anymore.Expected behavior
The possibility in the configuration to change the API server.
The text was updated successfully, but these errors were encountered: