mirror of
https://github.com/lldap/lldap.git
synced 2026-04-25 08:15:52 +03:00
[GH-ISSUE #870] [BUG] subpath don't work with image lldap/lldap:2024-03-07-debian|alpine #311
Labels
No labels
backend
blocked
bug
cleanup
dependencies
docker
documentation
duplicate
enhancement
enhancement
frontend
github_actions
good first issue
help wanted
help wanted
integration
invalid
ldap
pull-request
question
rust
rust
tests
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/lldap-lldap#311
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @zelogik on GitHub (Mar 18, 2024).
Original GitHub issue: https://github.com/lldap/lldap/issues/870
Describe the bug
subpath don't work as expected even with the commit/70d85524db4d7070f1fda499702bbbd3ec2dd8b5 with lldap/lldap:2024-03-07-debian|alpine
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Get the index.html from "app: make it possible to serve lldap behind a sub-path" | commit/70d85524db4d7070f1fda499702bbbd3ec2dd8b5
is the lldap/lldap:2024-03-07-debian|alpine is based on main branch ?
Logs
Not interesting
Additional context
To correct:
@martadinata666 commented on GitHub (Mar 18, 2024):
docker container use index_local.html as default rather than index.html due constraint that the js file doesn't load on an offline environment. https://github.com/lldap/lldap/tree/main/app there is also this thread https://github.com/lldap/lldap/issues/247 that may interest you
@zelogik commented on GitHub (Mar 18, 2024):
@martadinata666 : thanks, I understand why the file index_local.html exist now.
Already got a view on #247, and a quick reading on same-origin and CORS vulnerability/security.
But why many app still use subpath|dir ?
So there is no "clean way" to have the index.html (not the index_local.html) or that working version
With docker for the moment ?
@nitnelave commented on GitHub (Mar 18, 2024):
You can have the index.html locally, and then mount the file over the existing one when you create the container.
@zelogik commented on GitHub (Mar 18, 2024):
Yes of course, but it's what i call a "no clean way" to do it ;-)