Skip to content

fix: restrict OpenAPI references to local workdir files - #63

Draft
cloudcarver wants to merge 1 commit into
mainfrom
agent/security-openapi-local-refs
Draft

fix: restrict OpenAPI references to local workdir files#63
cloudcarver wants to merge 1 commit into
mainfrom
agent/security-openapi-local-refs

Conversation

@cloudcarver

Copy link
Copy Markdown
Owner

Summary

  • reject HTTP, HTTPS, and other non-file OpenAPI references
  • canonicalize local references and constrain them to the selected workdir
  • reject symlink escapes and non-regular reference targets
  • add tests proving remote servers are never contacted

Validation

  • go test ./...
  • go test -race ./pkg/openapi/bundle
  • go vet ./...

Closes #51

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.

[Security] Disable remote OpenAPI references by default

1 participant