Skip to content

Create folder_protection.csr#921

Merged
mgallien merged 3 commits intonextcloud:masterfrom
kreotropic:master
Mar 3, 2026
Merged

Create folder_protection.csr#921
mgallien merged 3 commits intonextcloud:masterfrom
kreotropic:master

Conversation

@kreotropic
Copy link
Copy Markdown
Contributor

Requesting certificate for folder_protection app. Repository: https://github.com/kreotropic/folder_protection

Signed-off-by: Ricardo Ferreira <rsfneg@gmail.com>
Copy link
Copy Markdown
Contributor

@mgallien mgallien left a comment

Choose a reason for hiding this comment

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

@kreotropic you should send us a certificate signing request instead of a certificate
please see the steps in the documentation
https://nextcloudappstore.readthedocs.io/en/latest/developer.html#obtaining-a-certificate

Correction to certificate request

Signed-off-by: Ricardo Ferreira <rsfneg@gmail.com>
@kreotropic
Copy link
Copy Markdown
Contributor Author

Updated the file with the CSR as requested. Thank you!

Copy link
Copy Markdown
Contributor

@mgallien mgallien left a comment

Choose a reason for hiding this comment

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

can you check again the certificate request ?
it should have as a common name (i.e. CN) the app id (i.e. folder_protection)

Recreated 

Signed-off-by: Ricardo Ferreira <rsfneg@gmail.com>
@kreotropic
Copy link
Copy Markdown
Contributor Author

Updated the CSR with the correct CN=folder_protection as requested.

mgallien added a commit that referenced this pull request Mar 3, 2026
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien mgallien merged commit bceb648 into nextcloud:master Mar 3, 2026
2 checks passed
@mgallien
Copy link
Copy Markdown
Contributor

mgallien commented Mar 3, 2026

@kreotropic you can find the new certificate here cd86858

@kreotropic
Copy link
Copy Markdown
Contributor Author

Thank you very much

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