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

CF Transactor(initial) #7037

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

CF Transactor(initial) #7037

wants to merge 8 commits into from

Conversation

haiodo
Copy link
Member

@haiodo haiodo commented Oct 25, 2024

CF Transactor(initial)

@haiodo haiodo requested a review from aonnikov October 25, 2024 10:02
@haiodo haiodo force-pushed the UBERF-7922p3 branch 3 times, most recently from d73373a to 1e90dd8 Compare October 29, 2024 06:47
@Chunosov Chunosov force-pushed the UBERF-7922p3 branch 2 times, most recently from b787f51 to 17205fb Compare December 13, 2024 12:21
# - https://developers.cloudflare.com/workers/wrangler/configuration/#environment-variables
# Note: Use secrets to store sensitive data.
# - https://developers.cloudflare.com/workers/configuration/secrets/
[vars]
Copy link
Member

Choose a reason for hiding this comment

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

please add dev/staging configs, or just staging with dev as default

workers/transactor/wrangler.toml Outdated Show resolved Hide resolved

setMetadata(serverPlugin.metadata.Secret, env.SERVER_SECRET)

console.log('Connecting DB to', env.HYPERDRIVE.connectionString)
Copy link
Member

Choose a reason for hiding this comment

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

nit: cloudflare recommends use objects for logging:

console.log({ message: 'Connecting DB to', hyperdrive: env.HYPERDRIVE.connectionString })

haiodo and others added 7 commits December 18, 2024 12:00
Signed-off-by: Andrey Sobolev <[email protected]>
Signed-off-by: Nikolay Chunosov <[email protected]>
Signed-off-by: Nikolay Chunosov <[email protected]>
Signed-off-by: Victor Ilyushchenko <[email protected]>
Signed-off-by: Andrey Sobolev <[email protected]>
Signed-off-by: Andrey Sobolev <[email protected]>
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.

4 participants