-
Notifications
You must be signed in to change notification settings - Fork 142
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
(ServiceException) json key is missing the type field #60
Comments
Could you please provide a full stack trace, and the contents of your config file? |
I use Google App Engine Before update my code work successfull. |
Fix logic if keyFile is null
The PR in question doesn't actually make any logical changes, it just changes the order of operations, so I cant see how this fixes your issue.. Please can you provide: The only way I can see this working is if |
Its same with my logic bro. If you learn clean code dont use === null in if argument |
Can you please update your config and replace
with
and let me know if it works for you then |
Can you tell me if |
i have solve it with this step :
|
@alfin87aa @ralfiannor Are you satisfied with this solution or do you need it to be able to work with a |
@nicja @MaggotMouth for me it's enough, thanks for your help 👍 |
Work perfect if i edit your code to handle null. Please, merge my pull request #61 and this issue will solve. |
Add google credential file in your source project is bad. Dont try it in production. |
You don't need to put it in your storage folder. |
@nicja this issue has wreaked havoc on us today. We need the ability to set I don't actually see why we need to do any |
@nicja @ralfiannor I rewrote the entire function so it makes more sense logically, it's also commented so you can easily tell what's going on. In my PR I wrote up the issue. |
I've got a same issue on standard app engine. It doesn't work with GOOGLE_CLOUD_KEY_FILE set to null. As I can recall, it used to work on v2.1.0, but it has "json key is missing the type field" error now. |
I encountered the same issue today, i see a pull request is awaiting. Can anyone check / approve this? |
Hi all, apologies for how long this has been outstanding. Unfortunately I can't test the specific scenarios that are broken. We now have 2 open PRs to fix this issue. I am in favour of #61 as it includes updates to the README. @cjcox17 @EntonoX @ralfiannor @shamsoft can you let me know if this PR solves the issue for you? Thanks! |
@nicja I tested #63 locally and within GCS docker container. I was not able to get it working without using the service account key as a secret. (So i needed to provide the 'key_file_path' environment variable). Without this environment variable i got a 'insufficient permission' error. I can not judge if this is a service account rights setting or some other problem. Maybe you or @cjcox17 can provide some details for this? |
🙏 I'm able to replicate this in GKE. I was able to hack around it by setting |
I'm using #63 in production without a key_file, I'll setup another instance this weekend to test. @EntonoX @matthewgoslett Can you provide your filesystem.php config for the GCS disk? I'll use this when I'm testing. |
Set up the newest version of this today on a new App Engine instance and ran into the same issue.
This solution worked for me, thank you @matthewgoslett |
@matthewgoslett @bredmor You are experiencing the ongoing issue that the pending pull request fixes. Pull request #63 Works in these cases. @ EntonoX @nicja I setup another Google Cloud Compute Engine Instance yesterday and verified the the pull request works as intended. @EntonoX Verify that you're using the newest config file included in the pull request and verify that your instance has the proper permissions on the service account. For testing I simply select "Allow full access to all API's". The README has been updated as well. Can you post your Laravel config, docker config, and a stack trace? @nicja If there is a separate issue for Docker, Kubernetes, or Google Cloud App Engine, I think it would make sense to approve the current pull request to fix the current issue and address the other issue separately so people like @matthewgoslett and @bredmor and myself don't continue to run into this issue. |
it's ok bro. |
I was deploying on GAE flexible environment and i got that error too, my config is like this: Has been found a solution? |
Downgrade version until fix version release. |
Hi found the solution... set : |
@nicja do you have an update on this issue? I'm getting it as well. Thanks! |
The empty string hack worked for me on the Kubernetes engine in GCP. |
I can confirm that it works on my workstation too. However, we really need to ability to set the |
This solution solved my error. Tks |
Is there anything we can do to get this fixed? Do you need any more information? As far as I can tell, there are numerous PRs addressing this issue and the issue itself is quite obvious as well IMHO. |
Add my PR to the list of ones fixing this issue. |
Setting an empty key worked for me. I added it to the env_variables part of app.yaml: |
Thank you! It worked. |
Muy Util, descague el json desde ahi y funcionó a la primera muchas gracias |
I updated to latest version 2.2.1 and get that if i upload image to bucket.
The text was updated successfully, but these errors were encountered: