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

fix: fixes the issue with websocket handshake and authz filter #1090

Closed
wants to merge 554 commits into from

Conversation

gsagula
Copy link
Contributor

@gsagula gsagula commented Jan 9, 2019

No description provided.

Flynn and others added 30 commits November 6, 2018 19:22
…release/0.50.0

* 'release/0.50.0' of github.com:datawire/ambassador:
  clean up sni docs
  edits to the SNI doc
* master:
  fix formatting error
  draft SNI documentation

# Conflicts:
#	docs/user-guide/sni.md
* master:
  add section on global tls config
  update ea versions
  Minor documentation clarifications
  Update bare-metal.md
  Added document for installing Ambassador on bare-metal kubernetes.
Signed-off-by: Gabriel <[email protected]>
* master:
  word tweak
  add new content on testing in prod
Arrange for "make clean" and "make venv" to play nicely
Signed-off-by: Gabriel <[email protected]>
Signed-off-by: Gabriel <[email protected]>
ext_authz: upgraded to the latest Envoy ext-authz with support to allowed authorization headers
* release/0.50.0:
  removed debug true from tests
  fixed v1 test and extended v0 tests
  deleted old test file
  code review changes
  wip: saving work
  improved auth tests and bumped envoy to the latest sha
  fixed the missing comma in the static whitelist
  added support for request and authorization whitelist headers
  initial ext_authz commit

# Conflicts:
#	ambassador/ambassador/ir/irauth.py
#	ambassador/tests/kat/test_ambassador.py
auth: added docs and rolled back v0 config logic
* release/0.50.0:
  added docs and rolled back v0 config logic

# Conflicts:
#	ambassador/ambassador/envoy/v2/v2listener.py
Flynn and others added 25 commits January 8, 2019 16:34
…make sure that rkeys include the object number
…' configs takes more than 3s on my Mac right now
Split service watching and config generation
* release/0.50.0:
  Quiet some debug logs.
  Lengthen the timeout on the diag service, since parsing the KAT tests' configs takes more than 3s on my Mac right now
  Update diagd to load configs generated by go-kubewatch.
  Switch to go-kubewatch 0.3.9 (for consistent configuration sets) and make sure that rkeys include the object number
  10-second warmup delay.
  Debugging plus fixes to create initial empty sync dir
  Switch to Go kubewatch 0.3.8
  CLI splitconfig can dump the IR as well.
  Quiet some logs
  SNAPSHOT: switch to split-config stuff. Some startup issues to untangle.
  Ditch dead cod.
  Sigh. How did this pass locally??
  Enable recursive config loading.
  It's really just the secret reader that needs to be composable -- the other resolution logic has to be held basically constant. This'll drastically simplify the split process.

# Conflicts:
#	ambassador/ambassador_diag/diagd.py
* release/0.50.0:
  Give Ambassador a way to post notices about, for example, deprecated config apiVersions. And use it.
* master:
  Update auth image
  Minor fixes
  minor typo fixes
  Final updates to multi-auth install
  Add proto to authservice
  Default AUTH_PROVIDER_URL set to https://example.auth0.com
  Minor fixes
  Minor fixes and comments to the template
  Fix RBAC
  Update template with Tenant and new Image
  Update SSO config in Ambassador Pro install docs
  minor fixes
  Added details about `scope`
  Updated doc for mutli domain auth. Included changes to access control doc
  Add documentation for configuring Auth0
  Changed documentation for configuring authentication with Ambassador Pro.
@kflynn
Copy link
Member

kflynn commented Jan 11, 2019

Let's not merge this into master. Already took #1091 into release/0.50.0. 😄

@kflynn kflynn closed this Jan 11, 2019
@kflynn kflynn deleted the websocket_fix branch January 11, 2019 19:58
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.

6 participants