Skip to content

Bump Duende IdentityServer (and update template typos)#876

Open
maartenba wants to merge 4 commits intodotnet:mainfrom
maartenba:main
Open

Bump Duende IdentityServer (and update template typos)#876
maartenba wants to merge 4 commits intodotnet:mainfrom
maartenba:main

Conversation

@maartenba
Copy link

Bump version numbers and update a few typos.

Copilot AI review requested due to automatic review settings August 14, 2025 13:41
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the project from IdentityServer4 to Duende IdentityServer, bumping the version and correcting related documentation and UI text. The changes include updating package versions, fixing typos, and updating references to point to the correct Duende repositories and documentation.

  • Bump Duende IdentityServer version from 7.1.1 to 7.3.0
  • Update UI text and links to reference Duende instead of IdentityServer4
  • Fix spelling error in logout confirmation message

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
Directory.Packages.props Updates DuendeVersion from 7.1.1 to 7.3.0
src/Identity.API/Views/Home/Index.cshtml Updates welcome message and links to reference Duende IdentityServer
src/Identity.API/Views/Account/Logout.cshtml Fixes typo "logut" to "logout"
src/Identity.API/Configuration/Config.cs Updates documentation URL to Duende's documentation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@DevTKSS
Copy link

DevTKSS commented Feb 8, 2026

@maartenba hey there, I recognized the old usings in the repo and found this PR of yours. Could you please add the eShop/src/ClientApp/* PackageReferences and usings to the updated contents of this PR for completeness? If it only updates the IdentityAPI but the ClientApp stays on the old reference, this might result in problems when breaking changes are introduced in newer package versions 👍

@maartenba
Copy link
Author

Sure thing! Force pushed an update getting these all to latest version @DevTKSS

@DevTKSS
Copy link

DevTKSS commented Feb 9, 2026

@maartenba
For the IdentityServer using parts I heared that this now requires any License (Community at least) but not sure if the sampling project or repo readme contains already the reference to those docs/websites where the user can get one.
Could you check?

@maartenba
Copy link
Author

Nothing in README I can see. Want me to add?

## Duende IdentityServer

The eShop application uses [Duende IdentityServer](https://duendesoftware.com/products/identityserver/), which is software that may require a license when deploying to a production environment. Depending on revenue, a free [community edition](https://duendesoftware.com/products/communityedition) is available for production use.

For development, testing, and general exploration, no license is needed.

Added information about Duende IdentityServer licensing requirements.
@maartenba
Copy link
Author

Added to the PR @DevTKSS

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