Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for url_style secret parameter #621

Merged
merged 7 commits into from
Feb 26, 2025

Conversation

Der-Henning
Copy link
Contributor

@Der-Henning Der-Henning commented Feb 21, 2025

This adds support for the URL_STYLE secret parameter.
Setting this parameter to path allows connecting to minio S3 storage.
Setting no value keeps the default behaviour and does not break the current behaviour.

This feature is requested in #151 and #207.

The code is working in my test environment.

As this is my first contribution to this project I would appreciate any feedback. Thank you for this awesome tool.

Copy link
Collaborator

@JelteF JelteF left a comment

Choose a reason for hiding this comment

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

Thanks! Some minor feedback, but overall looks good.

JelteF added a commit that referenced this pull request Feb 24, 2025
We need the necessary boilerplate for 0.4.0 development. In #621 the
contributor had problems understanding where to add their new SQL. This
should solve that problem.
@JelteF JelteF enabled auto-merge (squash) February 26, 2025 09:52
@JelteF JelteF merged commit a8425cb into duckdb:main Feb 26, 2025
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.

2 participants