Skip to content

Fix production-deployment README.md - #1180

Open
MosbyTheGreat wants to merge 1 commit into
sovity:version/10.4.1from
MosbyTheGreat:version/10.4.1
Open

Fix production-deployment README.md#1180
MosbyTheGreat wants to merge 1 commit into
sovity:version/10.4.1from
MosbyTheGreat:version/10.4.1

Conversation

@MosbyTheGreat

Copy link
Copy Markdown

The EDC_OAUTH_CLIENT_ID should not be the SKI/AKI, but instead the same as MY_EDC_PARTICIPANT_ID.

This PR fixes the question in the according README.md

Checklist

  • The PR title is short and expressive.
  • I have updated the CHANGELOG.md. Hints on formulation: Important information first, details second.
  • I have performed a self-review
  • I understand that my contributions will be contributed under the Contributor License Agreement (CLA) which has to be agreed to on your first contribution.

@CLAassistant

CLAassistant commented Aug 22, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@ghost

ghost commented Aug 27, 2025

Copy link
Copy Markdown

Hello and thank you for your contribution 🚀 I will forward it internally to decide how to proceed

@ghost

ghost commented Sep 3, 2025

Copy link
Copy Markdown

Hello, I will forward this internally, but the EDC_OAUTH_CLIENT_ID depends on the respective data space and how it is configured and setup. Some use older ski-aki, others IDs such as BPN or MDS-ID etc. As long as everyone within a data space follows the same schema after agreeing on an specific ID everything could actually be valid.

Here's a ping to the experts in this field: @richardtreier @ununhexium

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