Skip to content

PLAT-28 : Set secret_directory and extensions_directory setting for finer controls in duckdb#7712

Merged
begelundmuller merged 10 commits intomainfrom
disable_directory_access
Feb 27, 2026
Merged

PLAT-28 : Set secret_directory and extensions_directory setting for finer controls in duckdb#7712
begelundmuller merged 10 commits intomainfrom
disable_directory_access

Conversation

@k-anshul
Copy link
Member

@k-anshul k-anshul commented Aug 1, 2025

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@k-anshul k-anshul self-assigned this Aug 1, 2025
@k-anshul k-anshul requested a review from begelundmuller August 1, 2025 12:10
return err
}

// Find the instance dir for the data dir
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not ideal but at present there is no way to get instance dir from storage. The storage driver does not have concept of instance.

@k-anshul

This comment was marked as outdated.

@k-anshul k-anshul marked this pull request as draft August 1, 2025 12:33
@k-anshul k-anshul marked this pull request as ready for review February 11, 2026 10:24
@k-anshul k-anshul changed the title PLAT-28 : Disable enable_external_access for duckdb for finer controls PLAT-28 : Set secret_directory and extensions_directory setting for finer controls in duckdb Feb 27, 2026
@begelundmuller begelundmuller merged commit e395249 into main Feb 27, 2026
11 checks passed
@begelundmuller begelundmuller deleted the disable_directory_access branch February 27, 2026 13:35
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