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

Dockerfile for idp test app not using correct paths #1548

Closed
1 of 2 tasks
lullis opened this issue Jan 30, 2025 · 1 comment
Closed
1 of 2 tasks

Dockerfile for idp test app not using correct paths #1548

lullis opened this issue Jan 30, 2025 · 1 comment
Labels

Comments

@lullis
Copy link
Contributor

lullis commented Jan 30, 2025

Describe the bug

When building the idp/rdp test apps, the idp server fails when initiating login, as it can not find the registration/login.html template.

To Reproduce

  • Run docker-compose up -d --build to build the docker image and start the services.
  • Go to the rdp app (localhost:5173)
  • Click "login"
  • Results in Server Error

Expected behavior

  • The server should be able to find the login template and render the page properly.

Version

  • I have tested with the latest published release and it's still a problem.
  • I have tested with the master branch and it's still a problem.

Additional context
Is this "tests project" meant only for testing, or could/should it be used for development as well? It looks like no one is actually using during development (otherwise other people would spot these glaring errors), so perhaps I shouldn't be spending so much time in order to get this going.

@dopry
Copy link
Contributor

dopry commented Jan 31, 2025

@lullis The compose environment is meant for testing. We plan to setup playwright for end to end testing with it. We also intend to distribute the IDP image at some point in the future for people who want to stand up an IDP in their own container environment.

I opened #1550 with a more focused fix than the changes you proposed in #1549. Can you give it a go and let me if it resolves the issue you were having?

@lullis lullis closed this as completed Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants