Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Signature check #4

Open
wants to merge 2 commits into
base: development
Choose a base branch
from
Open

Signature check #4

wants to merge 2 commits into from

Conversation

whilefoo
Copy link
Contributor

@whilefoo whilefoo commented Jun 4, 2024

Currently the public key for the ubiquibot github app is retrieved from the env but later this should be a part of the SDK (or at least plugin starter template) so plugin developers don't have to worry about that

Copy link

github-actions bot commented Jun 4, 2024

Coverage Report (0%) 
File% Stmts% Branch% Funcs% LinesUncovered Line #s

@0x4007
Copy link
Member

0x4007 commented Jun 5, 2024

Can you make sure that CI passes? Alternatively make a new task for it.

@gentlementlegen
Copy link
Member

@0x4007 On it #5

@0x4007
Copy link
Member

0x4007 commented Jun 6, 2024

@0x4007 On it #5

Please merge this if safe to merge.

@gentlementlegen
Copy link
Member

Before merging, @whilefoo could you pass the the public key, or could you set it yourself the in Action Env? Thanks.

@whilefoo
Copy link
Contributor Author

whilefoo commented Jun 6, 2024

I'm not sure if I have access to the private key and I don't know which github app the new bot uses right now

@0x4007
Copy link
Member

0x4007 commented Jun 7, 2024

I'm not sure if I have access to the private key and I don't know which github app the new bot uses right now

Can try using @gentlementlegen v2 testing bot.

But yeah should be an env var

@gentlementlegen
Copy link
Member

Can be any dummy value as long as it works with the new bot and new plugins to get valid signatures, you can DM it to me and I'll update it if you want.

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