Skip to content
This repository has been archived by the owner on Jan 6, 2020. It is now read-only.

API Key not being found in Production #69

Closed
WildEgo opened this issue Oct 24, 2019 · 8 comments · Fixed by #70
Closed

API Key not being found in Production #69

WildEgo opened this issue Oct 24, 2019 · 8 comments · Fixed by #70
Assignees
Labels
status:complete This issue has been completed and is done type:bug This issue is a bug

Comments

@WildEgo
Copy link

WildEgo commented Oct 24, 2019

Describe the bug
The API Key is not found when in production (using laravel)

To Reproduce
Steps to reproduce the behavior:

  1. Just a normal call.

Expected behavior
Finding the key since the key is set on the .env file both the main .env as the .env in the folder itself

Screenshots
https://i.ibb.co/HV7jfKB/aaaaaa.png

Desktop (please complete the following information):

  • OS: CENTOS 7.4
  • PHP Version: 7.2
  • RiotQuest Version: 1.5.0 (can't update to latest since I'm using laravel 5.8)
@elreco
Copy link
Contributor

elreco commented Oct 24, 2019

Hello, try to delete this file

bootstrap/cache/config.php

@junlarsen junlarsen added the type:bug This issue is a bug label Oct 24, 2019
@junlarsen
Copy link
Owner

junlarsen commented Oct 24, 2019

Hi, if the above did not work, attempt to delete /vendor/supergrecko/riot-quest and composer update.

It looks like it is trying to pull src/storage/templates/summoner.json. The file is no longer named summoner.json which is why you're getting this error.

@WildEgo
Copy link
Author

WildEgo commented Oct 24, 2019

First didn't work composer update didn't work, update didn't update anything, any ideas?

@junlarsen
Copy link
Owner

Hi, I've figured out why it wasn't working. It appears that the engine uses strtolower on the file name. The reason it would work on my setup is because Windows ignores case in file names whereas Unix systems don't ignore it.

I'll be pushing a new release which fixes this issue asap.

@WildEgo
Copy link
Author

WildEgo commented Oct 24, 2019

Please be sure it works with laravel 5.8 or lower since ur current dependencies kill any chance of me updating it

@junlarsen
Copy link
Owner

Yeah, I'll work out the dependencies as well

@WildEgo
Copy link
Author

WildEgo commented Oct 24, 2019

Thanks man I really appreciate

junlarsen added a commit that referenced this issue Oct 24, 2019
Revert #66 - unwanted side effect
junlarsen added a commit that referenced this issue Oct 24, 2019
Fix #69 - remove strtolower() call on file path
Fix #66 - unwanted side effects in non-laravel envs
@junlarsen
Copy link
Owner

Pushed a new release, 1.5.2

Let me know if you still have problems with dependencies or anything by opening a new issue

@junlarsen junlarsen added the status:complete This issue has been completed and is done label Oct 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:complete This issue has been completed and is done type:bug This issue is a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants