Skip to content

fix: move auth middleware to connect#12

Merged
nadilas merged 6 commits intomasterfrom
fix/move-auth-middleware-to-connect
Mar 28, 2024
Merged

fix: move auth middleware to connect#12
nadilas merged 6 commits intomasterfrom
fix/move-auth-middleware-to-connect

Conversation

@nadilas
Copy link
Copy Markdown
Member

@nadilas nadilas commented Mar 28, 2024

No description provided.

Copy link
Copy Markdown
Member Author

nadilas commented Mar 28, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nadilas and the rest of your teammates on Graphite Graphite

Comment thread cli/cmd/config.go
rootCmd.PersistentFlags().String("addr", defaultConfig.BackendAddr, "The backend service address")
rootCmd.PersistentFlags().StringP("proxy", "p", defaultConfig.ReverseProxyAddr, "The reverse proxy which is directed at the module storage.")
rootCmd.PersistentFlags().BoolP("debug", "d", defaultConfig.Debug, "Toggles whether debug logs are enabled")
rootCmd.PersistentFlags().Bool("auth_enabled", defaultConfig.AuthEnabled, "Whether the backend service should use authentication")
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@waxb this new app arg line needs to add another startup argument in the infra

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

automatic deploy on dev and auto pr on any other environments?

@nadilas nadilas force-pushed the fix/move-auth-middleware-to-connect branch from 1288363 to 0f02039 Compare March 28, 2024 20:00
@nadilas nadilas merged commit 8a9fbe7 into master Mar 28, 2024
@nadilas nadilas deleted the fix/move-auth-middleware-to-connect branch March 28, 2024 20:00
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