-
Notifications
You must be signed in to change notification settings - Fork 273
feat(dashboard-api): supabase auth users sync background runner #2247
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
base: main
Are you sure you want to change the base?
Changes from 22 commits
c50d248
68a6265
9d8fe63
1e5f28b
56154df
3a431aa
eb69678
bb9fa39
461d2ec
ce0cbd1
4d622ce
84ecd06
3f56979
386d0c6
3e3fa3c
5e0972b
6b5a5c8
1113654
fb0afd4
bd4dd80
97e305a
c98878f
e314b96
db124a4
0e7f147
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,6 +13,10 @@ endif | |
| HOSTNAME := $(shell hostname 2> /dev/null || hostnamectl hostname 2> /dev/null) | ||
| $(if $(HOSTNAME),,$(error Failed to determine hostname: both 'hostname' and 'hostnamectl' failed)) | ||
|
|
||
| define DASHBOARD_API_EXTRA_ENV | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think it might have been simpler to expose the env variable directly rather than having it nested under extra env |
||
| $$(printf '%s' "$${DASHBOARD_API_ENV_VARS:-}" | jq -r '(if .=="" then empty elif type=="string" then (fromjson? // empty) else . end) | to_entries? // [] | map("\(.key)=\(.value|tostring|@sh)") | join(" ")') | ||
| endef | ||
|
|
||
| .PHONY: generate | ||
| generate: | ||
| go generate ./... | ||
|
|
@@ -33,12 +37,14 @@ build-and-upload: | |
| .PHONY: run | ||
| run: | ||
| make build | ||
| ./bin/dashboard-api | ||
| @EXTRA_ENV=$(DASHBOARD_API_EXTRA_ENV); \ | ||
| eval "env $$EXTRA_ENV ./bin/dashboard-api" | ||
|
|
||
| .PHONY: run-local | ||
| run-local: | ||
| make build | ||
| NODE_ID=$(HOSTNAME) ./bin/dashboard-api | ||
| @EXTRA_ENV=$(DASHBOARD_API_EXTRA_ENV); \ | ||
| eval "env NODE_ID=$(HOSTNAME) $$EXTRA_ENV ./bin/dashboard-api" | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is the usage of eval safe (secure) here? Can we avoid it? |
||
|
|
||
| .PHONY: test | ||
| test: | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.