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

CDK #543

Merged
merged 25 commits into from
Dec 7, 2023
Merged

CDK #543

merged 25 commits into from
Dec 7, 2023

Conversation

gmbronco
Copy link
Contributor

Changes:

  • node 18 EBS
  • moving sensitive ENV to secrets
  • making PROTOCOL and WORKER_QUEUE_URL env optional

@gmbronco gmbronco force-pushed the cdk branch 3 times, most recently from 869decd to ceedaeb Compare November 30, 2023 16:25
@gmbronco gmbronco force-pushed the cdk branch 3 times, most recently from 8098c99 to 88220f1 Compare December 1, 2023 11:33
@@ -63,7 +63,8 @@ async function runIfNotAlreadyRunning(

export function configureWorkerRoutes(app: Express) {
app.post('/', async (req, res, next) => {
const job = req.body as { name: string; chain: string };
const job = req.body as { name: string; chain: string; run?: boolean };
Copy link
Contributor

Choose a reason for hiding this comment

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

remove run? ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes!

@franzns franzns merged commit 93ed17c into v3-canary Dec 7, 2023
1 check passed
@franzns franzns deleted the cdk branch December 7, 2023 12:58
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