Skip to content

Commit

Permalink
fix: idp image can't find templates or statics (#1550)
Browse files Browse the repository at this point in the history
  • Loading branch information
dopry authored Jan 31, 2025
1 parent 48f4d54 commit 6d21bfb
Show file tree
Hide file tree
Showing 3 changed files with 27 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
* #1517 OP prompts for logout when no OP session
* #1512 client_secret not marked sensitive
* #1521 Fix 0012 migration loading access token table into memory
* #1584 Fix IDP container in docker compose environment could not find templates and static files.
<!--
### Security
-->
Expand Down
5 changes: 4 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,10 @@ ENV DATABASE_URL="sqlite:////data/db.sqlite3"
COPY --from=builder /opt/venv /opt/venv
ENV PATH="/opt/venv/bin:$PATH"
COPY --from=builder /code /code
RUN mkdir -p /code/tests/app/idp/static /code/tests/app/idp/templates
RUN mkdir -p /data/static /data/templates
COPY --from=builder /code/tests/app/idp/static /data/static
COPY --from=builder /code/tests/app/idp/templates /data/templates

WORKDIR /code/tests/app/idp
RUN apt-get update && apt-get install -y \
libpq5 \
Expand Down
22 changes: 22 additions & 0 deletions docs/contributing.rst
Original file line number Diff line number Diff line change
Expand Up @@ -325,6 +325,28 @@ Reviewing and Merging PRs

PRs that are incorrectly merged may (reluctantly) be reverted by the Project Leads.

End to End Testing
------------------

There is a demonstration Identity Provider (IDP) and Relying Party (RP) to allow for
end to end testing. They can be launched directly by following the instructions in
/test/apps/README.md or via docker compose. To launch via docker compose

.. code-block:: bash
# build the images with the current code
docker compose build
# wipe any existing services and volumes
docker compose rm -v
# start the services
docker compose up -d
Please verify the RP behaves as expected by logging in, reloading, and logging out.

open http://localhost:5173 in your browser and login with the following credentials:

username: superuser
password: password

Publishing a Release
--------------------
Expand Down

0 comments on commit 6d21bfb

Please sign in to comment.