Skip to content

doc: Add section on authorization on the Backstage Backend API#14

Merged
jhfgloria merged 3 commits into
PagerDuty:mainfrom
jkueh:main
Nov 5, 2025
Merged

doc: Add section on authorization on the Backstage Backend API#14
jhfgloria merged 3 commits into
PagerDuty:mainfrom
jkueh:main

Conversation

@jkueh

@jkueh jkueh commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

This PR adds user-facing documentation for PagerDuty/backstage-plugin-backend#114.

Comment on lines +169 to +172
```yaml
pagerduty:
disableUnauthenticatedAccess: true
```

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hello @jkueh! I reached out to you on another PR related to this matter. I think we should flip the enablement logic. It should be secure by default and less secure on demand. We would release a minor version to avoid people getting dragged by a changed like that on a patch version.

@jhfgloria jhfgloria left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me! Thanks so much for your contribution @jkueh!

@jhfgloria jhfgloria merged commit ac8f3fc into PagerDuty:main Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants