Skip to content

Commit

Permalink
Merge pull request #8016 from owncloud/fix-idp-config-dev-setup
Browse files Browse the repository at this point in the history
Fix idp config dev setup
  • Loading branch information
kulmann authored Nov 23, 2022
2 parents 224211d + 1e9a1d2 commit 82086f4
Show file tree
Hide file tree
Showing 8 changed files with 26 additions and 182 deletions.
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -60,3 +60,6 @@ tests/acceptance/report/cucumber_report.html

# third party licenses
/third-party-licenses

# dev setup certificates
/dev/docker/ocis-ca
Empty file added dev/docker/ocis-ca/.keep
Empty file.
14 changes: 14 additions & 0 deletions dev/docker/ocis.idp.config.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
clients:
- id: web
name: ownCloud web app
application_type: web
insecure: yes
trusted: yes
redirect_uris:
- https://host.docker.internal:9100/
- https://host.docker.internal:9200/
- https://host.docker.internal:9100/oidc-callback.html
- https://host.docker.internal:9200/oidc-callback.html
origins:
- https://host.docker.internal:9100
- https://host.docker.internal:9200
4 changes: 2 additions & 2 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ services:

# IDP
IDP_LDAP_BIND_PASSWORD: "${IDP_LDAP_BIND_PASSWORD:-some-ldap-idp-password}"
IDP_IDENTIFIER_REGISTRATION_CONF: ${IDP_IDENTIFIER_REGISTRATION_CONF:-/web/identifier-registration.yml}

# STORAGE
STORAGE_HOME_DRIVER: ${STORAGE_HOME_DRIVER:-ocis}
Expand All @@ -51,8 +50,9 @@ services:
PROXY_ENABLE_BASIC_AUTH: "${PROXY_ENABLE_BASIC_AUTH:-true}"
volumes:
- ./dev/docker/ocis.entrypoint.sh:/usr/bin/entrypoint
- ./dev/docker/ocis.idp.config.yaml:/etc/ocis/idp.yaml
- ./dev/docker/ocis-ca:/var/lib/ocis/proxy
- ./dist:/web/dist:ro
- ./tests/acceptance/mac-identifier-registration.yml:/web/identifier-registration.yml:ro
- ${OCIS_WEB_CONFIG:-./dev/docker/ocis.web.config.json}:/web/config.json:ro
extra_hosts:
- host.docker.internal:${DOCKER_HOST:-host-gateway}
Expand Down
157 changes: 0 additions & 157 deletions docs/testing/acceptance-tests-all.md

This file was deleted.

5 changes: 2 additions & 3 deletions docs/testing/testing.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
title: "Running acceptance tests"
title: "Running tests"
date: 2021-07-27T00:00:00+00:00
weight: 60
geekdocRepo: https://github.com/owncloud/web
Expand Down Expand Up @@ -135,8 +135,7 @@ to the nightwatch based e2e tests anymore.
In other words: only continue reading about our nightwatch based acceptance tests below if you need to debug a failing test.
{{< /hint >}}

At ownCloud, we have decided to adopt Docker as the main environment for developing our application.
This also applies for running our acceptance tests. To run the tests without Docker on your local machine, please refer to the [manual testing guide]({{< ref "acceptance-tests-all.md" >}})
At ownCloud, we have decided to adopt Docker as the main environment for developing our application. This also applies for running our acceptance tests.

#### Prerequisites

Expand Down
6 changes: 5 additions & 1 deletion rollup.config.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,11 @@ if (process.env.SERVER === 'true') {
serve({
host: '0.0.0.0',
contentBase: ['dist'],
port: process.env.PORT || 9100
port: process.env.PORT || 9100,
https: {
key: fs.readFileSync('./dev/docker/ocis-ca/server.key'),
cert: fs.readFileSync('./dev/docker/ocis-ca/server.crt')
},
})
)
plugins.push(
Expand Down
19 changes: 0 additions & 19 deletions tests/acceptance/mac-identifier-registration.yml

This file was deleted.

0 comments on commit 82086f4

Please sign in to comment.