Skip to content

feat: support nested structs, structured arrays, and better secret value handling in config #4727

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.

Sign up for GitHub

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

Merged
merged 16 commits into from
Oct 25, 2022
Merged

feat: support nested structs, structured arrays, and better secret value handling in config #4727

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
bb494dc
feat: support nested structs and arrays in config
f0ssel Oct 24, 2022
d84d377
update to main
f0ssel Oct 24, 2022
b1876bb
fix tests
f0ssel Oct 24, 2022
426e638
fix server
f0ssel Oct 24, 2022
faadaa0
fix ent
f0ssel Oct 24, 2022
4bcdd0a
get reflect working with ptr
f0ssel Oct 24, 2022
e6522cb
support slices
f0ssel Oct 24, 2022
429f91b
make tests pass
f0ssel Oct 24, 2022
552e438
move to default
f0ssel Oct 24, 2022
24bdf1b
safely return secrets
f0ssel Oct 24, 2022
d71ef7b
lol
f0ssel Oct 24, 2022
cead44f
fix tests
f0ssel Oct 24, 2022
c280692
lint
f0ssel Oct 24, 2022
a43640d
make gen
f0ssel Oct 24, 2022
1a37ac4
fis js
f0ssel Oct 24, 2022
b5bbd5f
make fmt
f0ssel Oct 24, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix ent
  • Loading branch information
@f0ssel
f0ssel committed Oct 24, 2022
commit faadaa0b2bf005bcd3ada041b988012c562aa134
8 changes: 4 additions & 4 deletions enterprise/cli/server.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,8 @@ import (
func server() *cobra.Command {
vip := deployment.NewViper()
cmd := agpl.Server(vip, func(ctx context.Context, options *agplcoderd.Options) (*agplcoderd.API, io.Closer, error) {
if options.DeploymentConfig.DERPServerRelayURL.Value != "" {
_, err := url.Parse(options.DeploymentConfig.DERPServerRelayURL.Value)
if options.DeploymentConfig.DERP.Server.RelayURL.Value != "" {
_, err := url.Parse(options.DeploymentConfig.DERP.Server.RelayURL.Value)
if err != nil {
return nil, nil, xerrors.Errorf("derp-server-relay-address must be a valid HTTP URL: %w", err)
}
Expand Down Expand Up @@ -63,8 +63,8 @@ func server() *cobra.Command {
SCIMAPIKey: []byte(options.DeploymentConfig.SCIMAPIKey.Value),
UserWorkspaceQuota: options.DeploymentConfig.UserWorkspaceQuota.Value,
RBAC: true,
DERPServerRelayAddress: options.DeploymentConfig.DERPServerRelayURL.Value,
DERPServerRegionID: options.DeploymentConfig.DERPServerRegionID.Value,
DERPServerRelayAddress: options.DeploymentConfig.DERP.Server.RelayURL.Value,
DERPServerRegionID: options.DeploymentConfig.DERP.Server.RegionID.Value,

Options: options,
}
Expand Down