Skip to content

fileserver: show symlink targets verbatim (#7476)#7579

Open
maxtruxa wants to merge 1 commit intocaddyserver:masterfrom
maxtruxa:reveal-links-as-is
Open

fileserver: show symlink targets verbatim (#7476)#7579
maxtruxa wants to merge 1 commit intocaddyserver:masterfrom
maxtruxa:reveal-links-as-is

Conversation

@maxtruxa
Copy link

See #7476

reveal_symlinks was exposing symlink targets as fully resolved absolute paths, even if the target is a relative path. With this change the link target is shown as-is, without resolving anything.

Example

tree output:

.
├── abs -> /home/maxtruxa/caddy-test-files/foo
├── foo
├── ind -> sub/rel
├── rel -> foo
└── sub
    └── rel -> ../foo

Before

caddy-before-1 caddy-before-2

After

caddy-after-1 caddy-after-2

Assistance Disclosure

No AI was used.

`reveal_symlinks` was exposing symlink targets as fully resolved absolute
paths, even if the target is a relative path. With this change the link target
is shown as-is, without resolving anything.
@CLAassistant
Copy link

CLAassistant commented Mar 18, 2026

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants