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

V8 support #34

Open
gsdias opened this issue May 28, 2019 · 6 comments
Open

V8 support #34

gsdias opened this issue May 28, 2019 · 6 comments

Comments

@gsdias
Copy link

gsdias commented May 28, 2019

Can I know when is the v8 being supported by this package? I think READ.ME is outdated as when implementing changes to Global.asax.cs it breaks and FileSystemProvider no longer exists in v8.

@ElijahGlover
Copy link
Owner

Hi @gsdias

There is no support for v8 yet, two reasons being the IFileSystem interface has changed slightly and new IoC infrastructure changes how custom IFileSystem implementations are registered. There is already work done on this in the v8-dev branch by but no means is it finished. Please feel free to lend a helping hand.

@gsdias
Copy link
Author

gsdias commented May 29, 2019 via email

@ElijahGlover
Copy link
Owner

I haven't had time to update this package to be compatible with v8 as yet. I'm familiar with @JimBobSquarePants Azure provider implementation, it should only take a few hours of full focus.

@gsdias
Copy link
Author

gsdias commented May 29, 2019 via email

@snerpton
Copy link

Also happy to do some testing when the time comes.

@neilgaietto
Copy link

@ElijahGlover
I needed this working for a new project so I got the bare minimum working here: https://github.com/neilgaietto/Umbraco-S3-Provider/tree/v8-dev-working

It sets up the component and composer which are the new features in 8.
Some issues that I did not resolve yet due to lack of time:

  1. Hostnames do not work because umbraco breaks when it tries to resolve a full url.
  2. Bucket prefix was having issue with slashes. Im sure this can be fixed but similar to the full url issue, umbraco does not like your path starting with no slash.
  3. I implemented this using app settings instead of parameters. So this version does not have updated instructions.
<add key="BucketFileSystem:Region" value="us-east-1" />
<add key="BucketFileSystem:BucketPrefix" value="media" />
<add key="BucketFileSystem:BucketName" value="{bucketname}" />
<add key="AWSAccessKey" value="" />
<add key="AWSSecretKey" value="" />

I will improve this if I get the time, but I'm hoping you get a chance to look at it too.
Thanks

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

No branches or pull requests

4 participants