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

Fixed use statement for Crypt in the importer, removed unused statements #15124

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

snipe
Copy link
Owner

@snipe snipe commented Jul 19, 2024

We had the older, short path for Crypt in the importer file, causing imports with encrypted custom fields to fail.

Signed-off-by: snipe <[email protected]>
Copy link

what-the-diff bot commented Jul 19, 2024

PR Summary

  • Removed Unnecessary Libraries
    The developer has removed some imported libraries (App\Models\AssetModel, Carbon\CarbonImmutable, Illuminate\Support\Facades\Auth, Carbon\Carbon, and Illuminate\Support\Facades\Log) in the AssetImporter.php file. This simplifies the code and helps reduce potential confusion or issues caused by unnecessary libraries.

  • Added Cryptography Library
    The Illuminate\Support\Facades\Crypt library has been imported into the AssetImporter.php file. This addition enhances the security features of the application by providing methods for encrypting and decrypting data.

@snipe snipe merged commit 32c7687 into develop Jul 19, 2024
8 of 9 checks passed
@snipe snipe deleted the fixes/path_for_crypt_in_importer branch July 19, 2024 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant