Skip to content

Add certificate signing request for learniq (renamed from scholiq) - #1197

Open
rubenvdlinde wants to merge 1 commit into
nextcloud:masterfrom
rubenvdlinde:csr/learniq
Open

Add certificate signing request for learniq (renamed from scholiq)#1197
rubenvdlinde wants to merge 1 commit into
nextcloud:masterfrom
rubenvdlinde:csr/learniq

Conversation

@rubenvdlinde

@rubenvdlinde rubenvdlinde commented Aug 27, 2026

Copy link
Copy Markdown

Hello, and thank you for looking after this repository. We know certificate requests are steady background work, and we are grateful for your time.

We are Conduction B.V. We build open source Nextcloud apps for Dutch public sector organisations, all EUPL-1.2.

Why we are asking

We renamed thirteen of our apps and we are taking them to production. We would like them signed and installable in time for the Nextcloud Conference in Berlin.

scholiq is now learniq.

Nextcloud checks that a certificate's CN matches the app id. Our scholiq certificate cannot sign learniq. So this is a new CSR rather than a reuse.

We would rather ask for a new certificate than keep signing with the old one. Signing does not check the CN, so a mismatched certificate produces a release that looks fine and fails occ integrity:check-app on the administrator's side. We would like to stop shipping that.

The old App Store entry

scholiq is still published on the App Store. It is superseded now, and we would like it taken down so there are not two entries for one app. We are happy to do that ourselves if there is a self-service route. If you would rather handle it, please tell us what you need from us.

The rest of the set

Thirteen apps are moving together. We opened one PR each, following the convention here:

We are glad to combine them into one PR if that is less work for you.

What we need

If the CSR is in order, merging it is everything we need from you. If anything should be split, renamed or resubmitted, tell us. We will send it the way you prefer.

@mgallien

Copy link
Copy Markdown
Contributor

@rubenvdlinde I would need you to also remove the certificate signing request from the old name in each PR
I would also need a confirmation by the original github account that requested the certificate that this is legitimate (I also understand that this is coming from ConductionNL)
The generation of new certificates and deletion of the obsolete ones will be handled by us
Please also check our documentation on what you need to do on such PR (see https://nextcloudappstore.readthedocs.io/en/latest/developer.html#obtaining-a-certificate).
As an example, the DCO CI check is invalid in this PR

@mgallien mgallien left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would need you to also remove the certificate signing request from the old name in each PR
I would also need a confirmation by the original github account that requested the certificate that this is legitimate (I also understand that this is coming from ConductionNL)
can you also please fix the failing DCO check ?
For more information, have a look at the failing DCO CI check from the PR
see also our documentation here https://nextcloudappstore.readthedocs.io/en/latest/developer.html#obtaining-a-certificate

setting the PR as needing changes for easier overview

The app id was renamed from scholiq to learniq. This adds the CSR for the new
id and removes the now-obsolete CSR for scholiq.

Signed-off-by: Ruben van der Linde <rubenvdlinde@gmail.com>
@rubenvdlinde

rubenvdlinde commented Aug 29, 2026

Copy link
Copy Markdown
Author

@rjzondervan could you please confirm this one?

This PR is part of the ConductionNL fleet rename: the app id scholiq becomes learniq. @mgallien asked for a confirmation from the GitHub account that originally requested the scholiq certificate that this rename is legitimate. You committed scholiq/scholiq.csr on 2026-06-03, so that is you. A short comment here confirming that the rename to learniq and the removal of the scholiq signing request are legitimate is all that is needed.

@mgallien I have addressed the other two points of your review:

  • the obsolete scholiq/scholiq.csr is now removed in this same PR (I left scholiq/scholiq.crt in place, since you indicated that the deletion of the obsolete certificates is handled by Nextcloud);
  • the commit now carries a Signed-off-by line, so the DCO check is green again.

@rjzondervan

Copy link
Copy Markdown
Contributor

Confirmed, I requested the original scholiq certificate; the rename to learniq and the removal of the old CSR are legitimate.

@rubenvdlinde

rubenvdlinde commented Aug 29, 2026

Copy link
Copy Markdown
Author

Ownership of these apps should sit with our corporate app store account

One additional point while this batch is being processed, relevant to all 13 PRs.

These are corporate applications built and maintained by Conduction, not personal side projects. Today the app store entries are owned by individual accounts (rubenvdlinde, rjzondervan, MWest2020). We want the ownership of the new app ids to sit with our corporate app store account conduction instead, so that:

  • ownership stays with the company rather than with individual employees, who may change role or leave;
  • these apps can be offered with enterprise support, which requires the company to be the accountable owner;
  • certificate renewals and any future transfer are handled through a single corporate account.

So our request is: when the certificates for the new ids are generated, please have the app store entries owned by conduction rather than by the individual accounts that requested the old ones. The 13 app ids in this batch are:

integriq, filinq, dossiq, stackiq, larpinq, thematiq, learniq, decidiq, buildiq, keepiq, humaniq, versioniq, planninq

The corresponding old ids to be retired are openconnector, docudesk, procest, softwarecatalog, larpingapp, nldesign, scholiq, decidesk, openbuild, doriath and app_versions (hrmq and planix were never published, so humaniq and planninq are first-time requests).

If anything is needed from our side to make that transfer possible, such as a confirmation from each of the current owner accounts or a different procedure entirely, please let us know and we will arrange it.

@rubenvdlinde

Copy link
Copy Markdown
Author

@mgallien i see that we can also mark them as orphaned ourselfs (kinda the layout doesnt quite fit) , but would that remove them from the app store?

image

@rubenvdlinde

Copy link
Copy Markdown
Author

@MWest2020 could you please confirm this one as well?

This is part of the same ConductionNL fleet rename batch: the app id scholiq becomes learniq. @mgallien asked for a confirmation from the GitHub account that originally requested the certificate, and you have already given it on #1204, #1205, #1206, #1208 and #1209. This PR is one of the remaining eight that still needs it.

For completeness on the other half of the request: this PR already removes scholiq/scholiq.csr and adds learniq/learniq.csr, so the old name is not left behind.

@rubenvdlinde

Copy link
Copy Markdown
Author

@mgallien all three points from your review are addressed. This PR is ready.

1. Old certificate signing request removed. This PR deletes scholiq/scholiq.csr and adds learniq/learniq.csr. The diff shows 26 deletions on the old path.

2. Confirmation by the original requester. @rjzondervan filed scholiq/scholiq.csr on 2026-06-03. He confirmed this rename here on 2026-08-29: "Confirmed, I requested the original scholiq certificate; the rename to learniq and the removal of the old CSR are legitimate."

3. DCO check. Green.

The PR is mergeable. Your review from 2026-08-27 still stands as "changes requested", which is the only thing blocking it now. I am re-requesting your review so it leaves your queue correctly.

This is one of seven PRs in the same ConductionNL rename batch: #1197, #1198, #1199, #1200, #1201, #1202, #1203. All seven are in the same state.

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.

3 participants