Skip to content

OIDC Documentation#100

Open
jmattheis wants to merge 3 commits into
masterfrom
oidc
Open

OIDC Documentation#100
jmattheis wants to merge 3 commits into
masterfrom
oidc

Conversation

@jmattheis

@jmattheis jmattheis commented Mar 29, 2026

Copy link
Copy Markdown
Member

See gotify/server#941 for more info about this.

Related MRs:

This was referenced Mar 29, 2026
@jmattheis

Copy link
Copy Markdown
Member Author

@eternal-flame-AD can you have a look at this too? This include the migration, config and oidc documentation.

Comment thread docs/config.md

:::

::: details 2.x Config

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Unless we intend to maintain the 2.x trunk after 3.x release this probably should go in the migration section (or removed in place of the migration CLI usage)

Comment thread docs/migrate-to-3.md

## API Changes

Gotify 3.x introduces [session elevation](./session-elevation.md). A client

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

"Introduces step up authentication via a session elevation timer" feels more self explanatory?

Comment thread docs/oidc.md

Only enable it if you trust that usernames in your identity provider map to the same people as your Gotify usernames.

## Authelia

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe put a "sample IdP config" section and out Authelia as a subsection

Comment thread docs/migrate-to-3.md
## CLI Changes

The binary now uses subcommands. You should migrate to using the `serve`
subcommand. For backwards compatibility running goverter without a command will

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The project name is wrong (?)

@eternal-flame-AD eternal-flame-AD left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the work! Just small editorial suggestions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants