-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Backstage update #771
base: main
Are you sure you want to change the base?
Backstage update #771
Changes from all commits
866847e
1d97ed0
a27e10b
aa4e12c
d9ae9e1
72c3644
9d1068a
22f8c86
1c64952
45cc39c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,3 @@ | ||
{ | ||
"version": "1.20.3" | ||
"version": "1.35.0" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,11 @@ | ||
import { createApp } from '@backstage/app-defaults'; | ||
import { AppRouter, FlatRoutes } from '@backstage/core-app-api'; | ||
import { AlertDisplay, OAuthRequestDialog } from '@backstage/core-components'; | ||
import { | ||
AlertDisplay, | ||
OAuthRequestDialog, | ||
SignInPage, | ||
} from '@backstage/core-components'; | ||
import { githubAuthApiRef } from '@backstage/core-plugin-api'; | ||
import { apiDocsPlugin } from '@backstage/plugin-api-docs'; | ||
import { | ||
CatalogEntityPage, | ||
|
@@ -39,6 +44,22 @@ import { searchPage } from './components/search/SearchPage'; | |
import { PrefixNavigate } from './components/utils/PrefixNavigate'; | ||
|
||
const app = createApp({ | ||
components: { | ||
SignInPage: props => ( | ||
<SignInPage | ||
{...props} | ||
providers={[ | ||
'guest', | ||
{ | ||
id: 'github-auth-provider', | ||
title: 'GitHub', | ||
message: 'Sign in using GitHub', | ||
apiRef: githubAuthApiRef, | ||
}, | ||
]} | ||
/> | ||
), | ||
Comment on lines
+48
to
+61
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Am I correct in thinking this shows up before the user is able to perform any other action? If so, can we somehow make There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes it does, you have to click Enter as guest user when first launched. I don't know if that is possible, but I will investigate. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Guests can go to We might need a custom auth resolver to automatically login as guest. This might also be a problem:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. A custom auth resolver sounds like a good call The shared guest identity isn't an issue for us as we run backstage in an immutable configuration. If you were to allow people to edit things in the UI it would certainly be an issue |
||
}, | ||
apis, | ||
bindRoutes({ bind }) { | ||
bind(catalogPlugin.externalRoutes, { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where does this come from?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is a fake secret because the field has to be provided, but the server is unauthenticated.