Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
62 commits
Select commit Hold shift + click to select a range
fa2bdda
Fix flaky tests, add debugging to others (#3041)
smatting Jan 31, 2023
f959352
Streamline team feature ExposeInvitationURLsToTeamAdminConfig impleme…
battermann Jan 31, 2023
0a8d0a4
charts: Mark all test resources to be only created while running test…
akshaymankar Jan 31, 2023
17b6e3a
Update searchability documentation (#3044)
smatting Feb 1, 2023
a54578a
play with hasserver instance
battermann Feb 1, 2023
b30fe2d
Add security response about wire.com DoS and HTML injection
sanojwr Feb 1, 2023
3373794
Update docs/src/security-responses/2023-01-19_html_injection.md
sanojwr Feb 1, 2023
d01cbb2
Update docs/src/security-responses/2023-01-19_html_injection.md
sanojwr Feb 1, 2023
15cfd83
Merge pull request #3050 from wireapp/sanojwr/security_responses
sanojwr Feb 1, 2023
e8408bf
.gitignore: Add third party modules of nginz
smatting Feb 2, 2023
70aa9a7
...
fisx Feb 2, 2023
afdbe5c
...
fisx Feb 2, 2023
d58d78a
...
fisx Feb 2, 2023
5087280
...
fisx Feb 2, 2023
a721f44
...
fisx Feb 2, 2023
633cfae
spar tests deflake: do an ES refresh, not reindex (#3048)
jschaul Feb 2, 2023
443f741
Helm hook type (#3049)
jschaul Feb 2, 2023
d182942
helmfile sync: speedup (#3052)
jschaul Feb 2, 2023
11e4167
Deflake metrics test (#3053)
jschaul Feb 2, 2023
6ac782b
WIP: attempt to use checkType in OAuth logic
pcapriotti Feb 2, 2023
58b2955
...
fisx Feb 2, 2023
be524fa
[FS-1075] Extend the Swagger documentation for federation error types…
Feb 2, 2023
e7724d7
wip
battermann Feb 2, 2023
cb62224
Test helper SQSWatcher: use purgeQueue (#3054)
jschaul Feb 2, 2023
7d1504b
Improve helm test output; and provide the means (even if disabled due…
jschaul Feb 2, 2023
3d7d9b2
clean up
battermann Feb 3, 2023
95dde99
more clear comments
battermann Feb 3, 2023
e4b69e8
clean up
battermann Feb 3, 2023
f37c319
another cleanup
battermann Feb 3, 2023
716b5f8
Small fixes to documentation (#3060)
smatting Feb 3, 2023
a0549ee
conn id optional
battermann Feb 3, 2023
f6e46c3
Lower the log level of federator inotify (#3056)
pcapriotti Feb 3, 2023
8051138
Update spar docs (#3038)
fisx Feb 3, 2023
ed226cd
Allow single scopes and lists of scopes in routes.
fisx Feb 3, 2023
7d735c6
Merge remote-tracking branch 'refs/remotes/origin/SQSERVICES-1885-be-…
fisx Feb 3, 2023
a7e4cbf
instance HasSwagger (ZAuthServant ...)
fisx Feb 3, 2023
70d1f47
Fixup
fisx Feb 3, 2023
e0c9c91
Merge remote-tracking branch 'origin/develop' into SQSERVICES-1885-be…
fisx Feb 6, 2023
d93bb7d
Tweak HasSwagger docs.
fisx Feb 6, 2023
1a6e02a
Merge branch 'SQSERVICES-1885-be-oauth-scopes-with-regard-to-calendar…
battermann Feb 6, 2023
3407a2f
support lenient, optional zoauth combinators.
fisx Feb 6, 2023
d7be8cc
Merge branch 'SQSERVICES-1885-be-oauth-scopes-with-regard-to-calendar…
battermann Feb 6, 2023
fea0a27
Fixup
fisx Feb 6, 2023
8ad2b3f
make oauth work with swagger
battermann Feb 6, 2023
15015d0
show scope(s) for each endpoint in swagger
battermann Feb 6, 2023
4eece1d
Merge branch 'SQSERVICES-1885-be-oauth-scopes-with-regard-to-calendar…
battermann Feb 6, 2023
8192d7e
fix
battermann Feb 6, 2023
0003255
fix 2
battermann Feb 6, 2023
a9f4d3e
grammar
battermann Feb 6, 2023
389fa39
make JWK available in galley
battermann Feb 7, 2023
fa47d21
oauth access to conversation create works
battermann Feb 7, 2023
26e75c5
jwk effect shared
battermann Feb 7, 2023
54a7cb2
generated local nix packages
battermann Feb 7, 2023
501a96f
setup config for secret for staging
battermann Feb 7, 2023
62da199
fix tests
battermann Feb 7, 2023
87e6dd8
enable oauth for post conversations
battermann Feb 7, 2023
9d5f5e3
renaming
battermann Feb 7, 2023
737ba2b
fix
battermann Feb 7, 2023
8d61c29
oauth scope for feature-configs
battermann Feb 8, 2023
5047395
wip
battermann Feb 8, 2023
fdff6a6
clean up
battermann Feb 8, 2023
0c14cca
update comment
battermann Feb 8, 2023
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
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -112,3 +112,6 @@ result-*

/integration-ca-key.pem
/integration-ca.pem

services/nginz/third_party/headers-more-nginx-module
services/nginz/third_party/nginx-module-vts
Comment on lines +116 to +117
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
services/nginz/third_party/headers-more-nginx-module
services/nginz/third_party/nginx-module-vts
/services/nginz/third_party/headers-more-nginx-module
/services/nginz/third_party/nginx-module-vts

7 changes: 4 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ elasticsearch-ephemeral minio-external cassandra-external \
nginx-ingress-controller nginx-ingress-services reaper sftd restund coturn \
inbucket k8ssandra-test-cluster
KIND_CLUSTER_NAME := wire-server
HELM_PARALLELISM ?= 1 # 1 for sequential tests; 6 for all-parallel tests
Copy link
Contributor

Choose a reason for hiding this comment

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

I think that should go to a separate PR, no?


package ?= all
EXE_SCHEMA := ./dist/$(package)-schema
Expand Down Expand Up @@ -315,15 +316,15 @@ kube-integration: kube-integration-setup kube-integration-test

.PHONY: kube-integration-setup
kube-integration-setup: charts-integration
export NAMESPACE=$(NAMESPACE); ./hack/bin/integration-setup-federation.sh
export NAMESPACE=$(NAMESPACE); export HELM_PARALLELISM=$(HELM_PARALLELISM); ./hack/bin/integration-setup-federation.sh

.PHONY: kube-integration-test
kube-integration-test:
export NAMESPACE=$(NAMESPACE); ./hack/bin/integration-test.sh
export NAMESPACE=$(NAMESPACE); export HELM_PARALLELISM=$(HELM_PARALLELISM); ./hack/bin/integration-test.sh

.PHONY: kube-integration-teardown
kube-integration-teardown:
export NAMESPACE=$(NAMESPACE); ./hack/bin/integration-teardown-federation.sh
export NAMESPACE=$(NAMESPACE); export HELM_PARALLELISM=$(HELM_PARALLELISM); ./hack/bin/integration-teardown-federation.sh

.PHONY: kube-integration-e2e-telepresence
kube-integration-e2e-telepresence:
Expand Down
1 change: 1 addition & 0 deletions changelog.d/4-docs/federation-error-type
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Extend the docs on the federation error type
1 change: 1 addition & 0 deletions changelog.d/4-docs/pr-3038
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Update SAML/SCIM docs
1 change: 1 addition & 0 deletions changelog.d/5-internal/deflake-metrics
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Deflake integration test: metrics
Copy link
Contributor

Choose a reason for hiding this comment

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

same here. am i reading the wrong diff?

1 change: 1 addition & 0 deletions changelog.d/5-internal/federator-log
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Lower the log level of federator inotify
1 change: 1 addition & 0 deletions changelog.d/5-internal/helm-setup
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
CI integration setup time should be reduced: tweak the way cassandra-ephemeral is started
1 change: 1 addition & 0 deletions changelog.d/5-internal/helm-test
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
charts: Mark all service/secret/configmap test resources to be re-created by defining them as helm hooks (#3037, #3049)
1 change: 1 addition & 0 deletions changelog.d/5-internal/parallel-helm-tests
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Add config to allow to run helm tests for different services in parallel; improve integration test output logs
5 changes: 4 additions & 1 deletion charts/brig/templates/tests/brig-integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ apiVersion: v1
kind: Service
metadata:
name: "brig-integration"
annotations:
"helm.sh/hook": post-install
"helm.sh/hook-delete-policy": before-hook-creation
labels:
app: brig-integration
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
Expand All @@ -19,7 +22,7 @@ kind: Pod
metadata:
name: "{{ .Release.Name }}-brig-integration"
annotations:
"helm.sh/hook": test-success
"helm.sh/hook": test
labels:
app: brig-integration
release: {{ .Release.Name }}
Expand Down
3 changes: 3 additions & 0 deletions charts/brig/templates/tests/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ apiVersion: v1
kind: ConfigMap
metadata:
name: "brig-integration"
annotations:
"helm.sh/hook": post-install
"helm.sh/hook-delete-policy": before-hook-creation
data:
integration.yaml: |
brig:
Expand Down
3 changes: 3 additions & 0 deletions charts/brig/templates/tests/nginz-service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,9 @@ apiVersion: v1
kind: Service
metadata:
name: nginz-integration-http
annotations:
"helm.sh/hook": post-install
"helm.sh/hook-delete-policy": before-hook-creation
spec:
type: ClusterIP
ports:
Expand Down
3 changes: 3 additions & 0 deletions charts/brig/templates/tests/secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ apiVersion: v1
kind: ConfigMap
metadata:
name: brig-integration-secrets
annotations:
"helm.sh/hook": post-install
"helm.sh/hook-delete-policy": before-hook-creation
data:
# These "secrets" are only used in tests and are therefore safe to be stored unencrypted
provider-privatekey.pem: |
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ kind: Pod
metadata:
name: "{{ .Release.Name }}-cargohold-integration"
annotations:
"helm.sh/hook": test-success
"helm.sh/hook": test
spec:
volumes:
- name: "cargohold-integration"
Expand Down
3 changes: 3 additions & 0 deletions charts/cargohold/templates/tests/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ apiVersion: v1
kind: ConfigMap
metadata:
name: "cargohold-integration"
annotations:
"helm.sh/hook": post-install
"helm.sh/hook-delete-policy": before-hook-creation
data:
integration.yaml: |
cargohold:
Expand Down
11 changes: 11 additions & 0 deletions charts/cassandra-ephemeral/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,3 +22,14 @@ cassandra-ephemeral:
seed_size: 1
max_heap_size: 2048M
heap_new_size: 1024M

livenessProbe:
initialDelaySeconds: 10
periodSeconds: 10
successThreshold: 1
failureThreshold: 15
readinessProbe:
initialDelaySeconds: 10
periodSeconds: 10
successThreshold: 1
failureThreshold: 15
3 changes: 3 additions & 0 deletions charts/federator/templates/tests/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ apiVersion: v1
kind: ConfigMap
metadata:
name: "federator-integration"
annotations:
"helm.sh/hook": post-install
"helm.sh/hook-delete-policy": before-hook-creation
data:
integration.yaml: |
federatorInternal:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ kind: Pod
metadata:
name: "{{ .Release.Name }}-federator-integration"
annotations:
"helm.sh/hook": test-success
"helm.sh/hook": test
spec:
volumes:
- name: "federator-integration"
Expand Down
3 changes: 3 additions & 0 deletions charts/galley/templates/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,9 @@ data:
{{- if .settings.disabledAPIVersions }}
disabledAPIVersions: {{ .settings.disabledAPIVersions }}
{{- end }}
{{- if $.Values.secrets.oauthPublicJwk }}
oauthPublicJwk: /etc/wire/galley/secrets/public_jwk_oauth.json
{{- end }}
{{- if .settings.featureFlags }}
featureFlags:
sso: {{ .settings.featureFlags.sso }}
Expand Down
4 changes: 2 additions & 2 deletions charts/galley/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ spec:
# An annotation of the configmap checksum ensures changes to the configmap cause a redeployment upon `helm upgrade`
checksum/configmap: {{ include (print .Template.BasePath "/configmap.yaml") . | sha256sum }}
checksum/aws-secret: {{ include (print .Template.BasePath "/aws-secret.yaml") . | sha256sum }}
checksum/mls-secret: {{ include (print .Template.BasePath "/mls-secret.yaml") . | sha256sum }}
checksum/secret: {{ include (print .Template.BasePath "/secret.yaml") . | sha256sum }}
spec:
serviceAccountName: {{ .Values.serviceAccount.name }}
volumes:
Expand All @@ -35,7 +35,7 @@ spec:
name: "galley"
- name: "galley-secrets"
secret:
secretName: "galley-mls"
secretName: "galley"
containers:
- name: galley
image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
kind: Secret
metadata:
name: galley-mls
name: galley
labels:
app: galley
chart: "{{ .Chart.Name }}-{{ .Chart.Version }}"
Expand All @@ -14,3 +14,6 @@ data:
removal_ed25519.pem: {{ .Values.secrets.mlsPrivateKeys.removal.ed25519 | b64enc | quote }}
{{- end -}}
{{- end -}}
{{- if .Values.secrets.oauthPublicJwk }}
public_jwk_oauth.json: {{ .Values.secrets.oauthPublicJwk | b64enc | quote }}
{{- end -}}
3 changes: 3 additions & 0 deletions charts/galley/templates/tests/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ apiVersion: v1
kind: ConfigMap
metadata:
name: "galley-integration"
annotations:
"helm.sh/hook": post-install
"helm.sh/hook-delete-policy": before-hook-creation
data:
integration.yaml: |
galley:
Expand Down
7 changes: 5 additions & 2 deletions charts/galley/templates/tests/galley-integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ apiVersion: v1
kind: Service
metadata:
name: "galley-integration"
annotations:
"helm.sh/hook": post-install
"helm.sh/hook-delete-policy": before-hook-creation
labels:
app: galley-integration
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
Expand All @@ -19,7 +22,7 @@ kind: Pod
metadata:
name: "{{ .Release.Name }}-galley-integration"
annotations:
"helm.sh/hook": test-success
"helm.sh/hook": test
labels:
app: galley-integration
release: {{ .Release.Name }}
Expand All @@ -36,7 +39,7 @@ spec:
name: "galley-integration-secrets"
- name: "galley-secrets"
secret:
secretName: "galley-mls"
secretName: "galley"
containers:
- name: integration
image: "{{ .Values.image.repository }}-integration:{{ .Values.image.tag }}"
Expand Down
3 changes: 3 additions & 0 deletions charts/galley/templates/tests/secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ apiVersion: v1
kind: ConfigMap
metadata:
name: galley-integration-secrets
annotations:
"helm.sh/hook": post-install
"helm.sh/hook-delete-policy": before-hook-creation
data:
# These "secrets" are only used in tests and are therefore safe to be stored unencrypted
provider-privatekey.pem: |
Expand Down
3 changes: 3 additions & 0 deletions charts/gundeck/templates/tests/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ apiVersion: v1
kind: ConfigMap
metadata:
name: "gundeck-integration"
annotations:
"helm.sh/hook": post-install
"helm.sh/hook-delete-policy": before-hook-creation
data:
integration.yaml: |
gundeck:
Expand Down
2 changes: 1 addition & 1 deletion charts/gundeck/templates/tests/gundeck-integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ kind: Pod
metadata:
name: "{{ .Release.Name }}-gundeck-integration"
annotations:
"helm.sh/hook": test-success
"helm.sh/hook": test
spec:
volumes:
- name: "gundeck-integration"
Expand Down
2 changes: 2 additions & 0 deletions charts/nginz/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -429,6 +429,7 @@ nginx_conf:
envs:
- all
doc: true
enable_oauth: true
- path: /legalhold/conversations/(.*)
envs:
- all
Expand Down Expand Up @@ -500,6 +501,7 @@ nginx_conf:
- path: /feature-configs(.*)
envs:
- all
enable_oauth: true
- path: /galley-api/swagger-ui
disable_zauth: true
envs:
Expand Down
3 changes: 3 additions & 0 deletions charts/spar/templates/tests/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ apiVersion: v1
kind: ConfigMap
metadata:
name: "spar-integration"
annotations:
"helm.sh/hook": post-install
"helm.sh/hook-delete-policy": before-hook-creation
data:
integration.yaml: |
brig:
Expand Down
2 changes: 1 addition & 1 deletion charts/spar/templates/tests/spar-integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ kind: Pod
metadata:
name: "{{ .Release.Name }}-spar-integration"
annotations:
"helm.sh/hook": test-success
"helm.sh/hook": test
labels:
app: spar-integration
release: {{ .Release.Name }}
Expand Down
10 changes: 10 additions & 0 deletions docs/src/developer/reference/config-options.md
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,16 @@ certificate, is to run the following command:
openssl req -nodes -newkey ed25519 -keyout ed25519.pem -out /dev/null -subj /
```

### Public JWK for OAuth

Set the path to the public JWK key for OAuth like this:

```yml
# [galley.yaml]
settings:
oauthPublicJwk: test/resources/oauth/ed25519_public_jwk.json
```

## Feature flags

> Also see [Wire docs](https://docs.wire.com/how-to/install/team-feature-settings.html) where some of the feature flags are documented from an operations point of view.
Expand Down
Loading