Skip to content

Commit

Permalink
Fix #42546
Browse files Browse the repository at this point in the history
  • Loading branch information
@Sanne
Sanne committed Aug 14, 2024
1 parent 35f4c4a commit cc7bc0b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
Binary file modified docs/src/main/asciidoc/images/oidc-github-1.png
View file
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 1 addition & 1 deletion docs/src/main/asciidoc/security-openid-connect-providers.adoc
View file
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,7 @@ In order to set up OIDC for GitHub you need to create a new OAuth application in

image::oidc-github-1.png[role="thumb"]

Check warning on line 208 in docs/src/main/asciidoc/security-openid-connect-providers.adoc

View workflow job for this annotation

GitHub Actions / Linting with Vale 

[vale] reported by reviewdog 🐶
[Quarkus.Spelling] Use correct American English spelling. Did you really mean 'oidc'?

Raw Output:
{"message": "[Quarkus.Spelling] Use correct American English spelling. Did you really mean 'oidc'?", "location": {"path": "docs/src/main/asciidoc/security-openid-connect-providers.adoc", "range": {"start": {"line": 208, "column": 8}}}, "severity": "WARNING"}

Make sure to fill in the appropriate details, but more importantly the Authorization Callback URL, set to `http://localhost:8080/_renarde/security/github-success`
Make sure to fill in the appropriate details, but more importantly the Authorization Callback URL, set to `http://localhost:8080/_renarde/security/oidc-success`

Check warning on line 210 in docs/src/main/asciidoc/security-openid-connect-providers.adoc

View workflow job for this annotation

GitHub Actions / Linting with Vale 

[vale] reported by reviewdog 🐶
[Quarkus.TermsWarnings] Consider using 'verify' rather than 'Make sure' unless updating existing content that uses the term.

Raw Output:
{"message": "[Quarkus.TermsWarnings] Consider using 'verify' rather than 'Make sure' unless updating existing content that uses the term.", "location": {"path": "docs/src/main/asciidoc/security-openid-connect-providers.adoc", "range": {"start": {"line": 210, "column": 1}}}, "severity": "WARNING"}
(if you intend to test this using the Quarkus dev mode).

Now click on `Register application` and you'll be shown your application page:
Expand Down

0 comments on commit cc7bc0b

Please sign in to comment.