Skip to content

Commit

Permalink
only allow --oauth-provider for 'zrok reserve public'; changelog (#421)
Browse files Browse the repository at this point in the history
  • Loading branch information
michaelquigley committed Oct 25, 2023
1 parent 9f018e7 commit aba9f68
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
# v0.4.11

FIX: Include `--oauth-provider` and associated flags for the `zrok reserve` command, allowing reserved shares to specify OAuth authentication (https://github.com/openziti/zrok/issues/421)

# v0.4.10

CHANGE: The public frontend configuration has been bumped from `v: 2` to `v: 3`. The `redirect_host`, `redirect_port` and `redirect_http_only` parameters have been removed. These three configuration options have been replaced with `bind_address`, `redirect_url` and `cookie_domain`. See the OAuth configuration guide at `docs/guides/self-hosting/oauth/configuring-oauth.md` for more details (https://github.com/openziti/zrok/issues/411)
Expand Down
3 changes: 3 additions & 0 deletions cmd/zrok/reserve.go
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,9 @@ func (cmd *reserveCommand) run(_ *cobra.Command, args []string) {
req.Frontends = cmd.frontendSelection
}
if cmd.oauthProvider != "" {
if shareMode != sdk.PublicShareMode {
tui.Error("--oauth-provider only supported for public shares", nil)
}
req.OauthProvider = cmd.oauthProvider
req.OauthEmailDomains = cmd.oauthEmailDomains
req.OauthAuthorizationCheckInterval = cmd.oauthCheckInterval
Expand Down

0 comments on commit aba9f68

Please sign in to comment.