Add certificate signing request for humaniq (renamed from hrmq) - #1201
Add certificate signing request for humaniq (renamed from hrmq)#1201rubenvdlinde wants to merge 1 commit into
Conversation
mgallien
left a comment
There was a problem hiding this comment.
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
Signed-off-by: Ruben van der Linde <rubenvdlinde@gmail.com>
7a1956d to
54039a0
Compare
|
@mgallien I have addressed your review:
For the same reason there is no earlier requesting account that could confirm it. The request comes from ConductionNL, same as the other 12 PRs in this batch. |
|
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 (
So our request is: when the certificates for the new ids are generated, please have the app store entries owned by
The corresponding old ids to be retired are 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. |
|
@MWest2020 could you please confirm this one as well? This is part of the same ConductionNL fleet rename batch, for the app id On the other half of the request, removing the old name: there is nothing to remove here. |
|
@mgallien all three points from your review are addressed. This PR is ready. 1. Old certificate signing request removed. Nothing to remove here. This app has no earlier certificate in this repository: there is no 2. Confirmation by the original requester. There is no earlier request for this app, so there is no prior requester to confirm. The five renames in this batch that do replace an existing certificate were confirmed by @rjzondervan on 2026-08-29, and he filed each of those originals. 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. |
|
@mgallien on the two review points that do not apply to this PR, with the check I ran so you can redo it. There is no old certificate to remove. There is likewise no original requesting account to confirm. Since no certificate was ever issued for The DCO check passes. The same applies to #1203 (planninq, no |
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.
hrmqis nowhumaniq.Nextcloud checks that a certificate's CN matches the app id. Our
hrmqcertificate cannot signhumaniq. 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-appon the administrator's side. We would like to stop shipping that.The old App Store entry
hrmqwas never published, so there is nothing to take down for this one.The rest of the set
Thirteen apps are moving together. We opened one PR each, following the convention here:
openconnectorbecomesintegriqdocudeskbecomesfilinqprocestbecomesdossiqsoftwarecatalogbecomesstackiqlarpingappbecomeslarpinqnldesignbecomesthematiqscholiqbecomeslearniqdecideskbecomesdecidiqopenbuildbecomesbuildiqdoriathbecomeskeepiqhrmqbecomeshumaniq← this oneapp_versionsbecomesversioniqplanixbecomesplanninqWe 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.