Skip to content
This repository has been archived by the owner on Jun 12, 2018. It is now read-only.

Should create a default config file #5

Closed
iflamed opened this issue Jan 31, 2018 · 4 comments
Closed

Should create a default config file #5

iflamed opened this issue Jan 31, 2018 · 4 comments

Comments

@iflamed
Copy link

iflamed commented Jan 31, 2018

$salt = config('hashslug.appsalt', env('APP_KEY')) . $modelSalt;

The env function will work at laravel debug environment but not work when laravel running at production.

@iflamed
Copy link
Author

iflamed commented Jan 31, 2018

<?php

return [
    'appsalt' => env('APP_KEY')
];

This code will compatible with the history releases.

@balping
Copy link
Owner

balping commented Jan 31, 2018

I just tested it with APP_ENV=production and still works. The env function doesn't require local environment.

@iflamed
Copy link
Author

iflamed commented Jan 31, 2018

In production mode, The env function will not work after run php artisan config:cache
image

balping added a commit that referenced this issue Jan 31, 2018
@balping
Copy link
Owner

balping commented Jan 31, 2018

Thanks for pointing this out. The fix is released.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants