Skip to content

fix(gopher): block symlink escapes#85

Merged
ralyodio merged 1 commit into
profullstack:mainfrom
rissrice2105-agent:fix/gopher-symlink-escape
Jul 10, 2026
Merged

fix(gopher): block symlink escapes#85
ralyodio merged 1 commit into
profullstack:mainfrom
rissrice2105-agent:fix/gopher-symlink-escape

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Summary

  • resolve both the member public base and requested target with filepath.EvalSymlinks
  • reject Gopher selectors whose resolved target escapes the member area
  • add a regression test for a symlink pointing from public_html to a sibling secret file

Closes #84

Testing

  • go test ./internal/gopher

@ralyodio ralyodio merged commit c2a06f1 into profullstack:main Jul 10, 2026
5 checks passed
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.

Bug: Gopher public file resolver follows symlink escapes

2 participants