Skip to content

Moved data protection info to a new page in the General Information#1046

Merged
maartenba merged 13 commits intomainfrom
ar-data-protection-for-is-and-bff-and-more
Apr 1, 2026
Merged

Moved data protection info to a new page in the General Information#1046
maartenba merged 13 commits intomainfrom
ar-data-protection-for-is-and-bff-and-more

Conversation

@ProgrammerAL
Copy link
Copy Markdown
Contributor

@ProgrammerAL ProgrammerAL commented Mar 26, 2026

Fixes #471

  • Moved data protection info to a new page in the General Information section
    • Added callouts for BFF
  • Updated the Data Protection info for IdentityServer to link to the general topic.

…ection. Added callouts for BFF. Updated the Data Protection info for IdentityServer to link to the general topic.
@ProgrammerAL ProgrammerAL self-assigned this Mar 26, 2026
@ProgrammerAL ProgrammerAL added the documentation Improvements or additions to documentation label Mar 26, 2026
@ProgrammerAL
Copy link
Copy Markdown
Contributor Author

I moved the Data Protection content to the /general section because it's shared. We probably should have discussed ahead of time, but I didn't think there was anywhere else the docs could go. If anyone has a better suggestion, I'm open to it.

Copy link
Copy Markdown
Member

@maartenba maartenba left a comment

Choose a reason for hiding this comment

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

Left a couple of suggested changes.

There are some sections that seem to fit deployment/index.md better as they are IdentityServer-specific. Would consider moving those back.

Comment thread astro/src/content/docs/identityserver/deployment/index.md Outdated
Comment thread astro/src/content/docs/identityserver/deployment/index.md Outdated
Comment thread astro/src/content/docs/general/data-protection.md Outdated
Comment thread astro/src/content/docs/general/data-protection.md Outdated
Comment thread astro/src/content/docs/general/data-protection.md Outdated
Comment thread astro/src/content/docs/general/data-protection.md Outdated
Comment thread astro/src/content/docs/general/data-protection.md Outdated
Comment thread astro/src/content/docs/general/data-protection.md Outdated
Comment thread astro/src/content/docs/general/data-protection.md Outdated
Comment thread astro/src/content/docs/general/data-protection.md Outdated
ProgrammerAL and others added 7 commits March 26, 2026 17:39
Co-authored-by: Maarten Balliauw <maarten.balliauw@duendesoftware.com>
Co-authored-by: Maarten Balliauw <maarten.balliauw@duendesoftware.com>
Co-authored-by: Maarten Balliauw <maarten.balliauw@duendesoftware.com>
Co-authored-by: Maarten Balliauw <maarten.balliauw@duendesoftware.com>
Co-authored-by: Maarten Balliauw <maarten.balliauw@duendesoftware.com>
Co-authored-by: Maarten Balliauw <maarten.balliauw@duendesoftware.com>
Co-authored-by: Maarten Balliauw <maarten.balliauw@duendesoftware.com>
Comment thread astro/src/content/docs/general/data-protection.md Outdated
Comment thread astro/src/content/docs/general/data-protection.md Outdated
@wcabus
Copy link
Copy Markdown
Member

wcabus commented Mar 27, 2026

I still want to link to the overview of Data Protection defaults (https://learn.microsoft.com/en-us/aspnet/core/security/data-protection/configuration/default-settings), but don't know at the moment where it would be best to put this.

I don't believe we should repeat Microsoft's docs concerning the defaults however, in case they get updated with new scenarios. There is one exception imo: the use of DPAPI. If you rely on the default configuration on Windows-based environments, DPAPI or DPAPI-NG is being used over which you have no control, making it harder to migrate data protection keys.

ProgrammerAL and others added 2 commits March 31, 2026 11:21
Co-authored-by: Maarten Balliauw <maarten.balliauw@duendesoftware.com>
Co-authored-by: Wesley Cabus <wesley@gotsharp.be>
@ProgrammerAL
Copy link
Copy Markdown
Contributor Author

@wcabus

I still want to link to the overview of Data Protection defaults (https://learn.microsoft.com/en-us/aspnet/core/security/data-protection/configuration/default-settings), but don't know at the moment where it would be best to put this.

I don't believe we should repeat Microsoft's docs concerning the defaults however, in case they get updated with new scenarios. There is one exception imo: the use of DPAPI. If you rely on the default configuration on Windows-based environments, DPAPI or DPAPI-NG is being used over which you have no control, making it harder to migrate data protection keys.

That link is already in the ASP.NET Core Data Protection section.

I tried writing something up for DPAPI, but couldn't settle on the wording, or where to put it. I'll leave it out of this PR. But if you think of a good way toa dd it in, we can do it here.

ProgrammerAL and others added 3 commits March 31, 2026 14:26
…page, specifically the IdentityServer specific content. Also some rewording for simplicity or spelling fixes.
@ProgrammerAL ProgrammerAL requested a review from maartenba April 1, 2026 13:35
@wcabus
Copy link
Copy Markdown
Member

wcabus commented Apr 1, 2026

@wcabus

I still want to link to the overview of Data Protection defaults (https://learn.microsoft.com/en-us/aspnet/core/security/data-protection/configuration/default-settings), but don't know at the moment where it would be best to put this.
I don't believe we should repeat Microsoft's docs concerning the defaults however, in case they get updated with new scenarios. There is one exception imo: the use of DPAPI. If you rely on the default configuration on Windows-based environments, DPAPI or DPAPI-NG is being used over which you have no control, making it harder to migrate data protection keys.

That link is already in the ASP.NET Core Data Protection section.

I tried writing something up for DPAPI, but couldn't settle on the wording, or where to put it. I'll leave it out of this PR. But if you think of a good way toa dd it in, we can do it here.

Let's leave it like this for now. We already added more docs, links to MS Learn, and a shoutout to use escrow which basically would solve the DPAPI problem.

@ProgrammerAL ProgrammerAL requested a review from wcabus April 1, 2026 15:14
@maartenba maartenba merged commit f4a61e8 into main Apr 1, 2026
5 of 7 checks passed
@maartenba maartenba deleted the ar-data-protection-for-is-and-bff-and-more branch April 1, 2026 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expand Data Protection Docs to cover clients/BFF

3 participants