-
Notifications
You must be signed in to change notification settings - Fork 19
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
Secure https plugins #48
Conversation
Glad to see that you are pushing code again. |
@whilefoo Thank you for this. Will you take care of implementing it in the relevant plugins as well? |
Co-authored-by: Mentlegen <[email protected]>
Yes I have implemented it in |
I believe that https://github.com/ubiquibot/comment-incentives and https://github.com/ubiquibot/user-activity-watcher would remain as Actions because they might be long running tasks. Otherwise since all the others are handling commands yes I believe they can all be Workers (and I think it's already the case). Side note, plugins can even be hybrids with both Worker and Action if that is ever necessary someday. |
Resolves #43