revoke summarai.crt

[kyteinsky]

No description provided.

[nickvergessen]

needs revocation as usual

[kyteinsky]

Hi, sorry I don't understand what I should do, if it was meant for me? Also, the previous cert/key was not used to register any app.

[nickvergessen]

No, that's for Matthieu or the sysadmin doing it :P

[camilasan]

needs revocation as usual

if I run docker compose run php ./bin/certmanager certificate:revoke 728, I get The pull request need to have only a file: the certificate file.

Is that correct?

[nickvergessen]

I think you need to revoke 684 which is the original PR: #684

[vitormattos]

The revoke command was projected to revoke an appid.

At this PR have two actions: revoke and create a new cert.

To work with this tool, will be necessary two PR (the quick way, a PR revoking and a PR adding a new app). To use this PR will be necessary implement a new command, i.e. certificate:rename to consider 3 files changed by PR.

[kyteinsky]

I can convert this PR to just add summary_bot if you wish to revoke #684

[camilasan]

I think you need to revoke 684 which is the original PR: #684

this throws this error: This PR need to modify a csr file. The status of this PR is: added

[nickvergessen]

I can convert this PR to just add summary_bot if you wish to revoke #684

Might be easier to do both things in separate PRs:

1. Remove summarai in this PR and revoke it
2. New PR to request summary_bot

[vitormattos]

This PR need to modify a csr file. The status of this PR is: added

To revoke a csr, is necessary a change of csr file. The PR 684 is adding a new csr file and this already was made. The PR 684 can't be used to revoke a certificate.

A way that solve this is:

• change this PR to have only a file creating the summari_bot to unlock the flow to generate the crt file
• Create a new PR only deleting (revoke) the summari.crt

[kyteinsky]

summary_bot.csr is here: #729

[kyteinsky]

kind pingu

[camilasan]

when trying to revoke this PR with certificate:revoke 728, I am getting:
The pull request need to have only a file: the certificate file

Maybe the certificate file was not supposed to be deleted in this PR?

[camilasan]

@kyteinsky could you also remove the .csr file?

[kyteinsky]

And the folder too?

[camilasan]

And the folder too?

just the files.