soarecostin / file-vault

A Laravel package for encrypting and decrypting files of any size
MIT License
185 stars 62 forks source link

Laravel 9 incompatibility #24

Open juranir opened 2 years ago

juranir commented 2 years ago

I'm trying to install on Laravel 9 but I'm stuck, it seems incompatible. Someone are using on v9?

- Root composer.json requires soarecostin/file-vault ^1.2 -> satisfiable by soarecostin/file-vault[v1.2.0]. - soarecostin/file-vault v1.2.0 requires illuminate/support 5.8.* || 6.*.* || ^7.0 || ^8.0 -> found illuminate/support[v5.8.0, ..., 5.8.x-dev, v6.0.0, ..., 6.x-dev, v7.0.0, ..., 7.x-dev, v8.0.0, ..., 8.x-dev] but these were not loaded, likely because it conflicts with another require.

niekp commented 2 years ago

I've created a PR to update to laravel 9. https://github.com/soarecostin/file-vault/pull/25

Rubens10010 commented 2 years ago

I also neeed this. Is this package going deprecated? Almost two years now with no release

niekp commented 2 years ago

We've published a fork with Laravel 9 support: https://packagist.org/packages/brainstud/file-vault

gbalduzzi commented 1 year ago

Hi @niekp , thank for your commit! Do you use your fork with laravel 9 and S3? Because I'm getting some errors due to some incompatibility with the v3 FilesystemAdapters

Specifically, the FileVault.php around line 190:

protected function registerServices()
    {
        $this->setAdapter();

        if ($this->isS3File()) {
            $client = $this->adapter->getClient(); // <- Get Client not available anymore
            $client->registerStreamWrapper();
        }
    }
gbalduzzi commented 1 year ago

If anyone needs support for S3 using Laravel 9, I solved the incompatibility with this commit: https://github.com/tiknil/file-vault/commit/6b2cde9314d3d6e012be4d2c04b6a3f1dc465f64

tonimager commented 1 year ago

Any chance of making brainstud/file-vault compatible with Laravel 10?

Root composer.json requires brainstud/file-vault ^2.0 -> satisfiable by brainstud/file-vault[v2.0.0].
Conclusion: don't install laravel/framework v10.0.2 (conflict analysis result)
brainstud/file-vault   v2.0.0   requires   illuminate/support (^8.0 || ^9.0 )
spatie/laravel-ignition   1.6.4   requires   illuminate/support (^8.77|^9.27)