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

[10.x] Support inline disk for scoped driver #47776

Merged
merged 3 commits into from
Jul 24, 2023
Merged

[10.x] Support inline disk for scoped driver #47776

merged 3 commits into from
Jul 24, 2023

Conversation

alexbowers
Copy link
Contributor

Currently, if you want to use the scoped disk driver, there is a requirement to have a separate disk that is unscoped.

If you want to make all of your storage writes in a scoped manner (for example, to customer specific directories), then this requires you to have another disk that is configured, that you don't want to ever actually use directly.

There is potential for a developer to use the disk inadvertently, or for you to have to mark it as 'hidden' in some way, such as prefixing with an underscore, or similar.

This pull request allows you to optionally declare your disk inline in the scoped driver, so that you don't need to have this additional storage disk, if you only ever want to interact with the scoped driver itself.

@alexbowers alexbowers changed the title Support inline disk for scoped driver [10.x] Support inline disk for scoped driver Jul 19, 2023
@alexbowers alexbowers marked this pull request as ready for review July 19, 2023 11:12
@alexbowers
Copy link
Contributor Author

That SQL Server 2019 failure is strange, not sure how to resolve that. is it a known isuse?

@driesvints
Copy link
Member

@alexbowers can you rebase? I disabled it for now.

@alexbowers
Copy link
Contributor Author

Done

@taylorotwell taylorotwell merged commit d44b529 into laravel:10.x Jul 24, 2023
@alexbowers alexbowers deleted the patch-2 branch July 24, 2023 15:56
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.

3 participants