-
Notifications
You must be signed in to change notification settings - Fork 1
Running on App Engine #3
Comments
Hello @kubaraczkowski, thanks for being interrested on to my project! I didn't thought someone would find it usefull so I let him be for a moment. Actually there is some flaws you need to consider and I would like to fix those before First, I am using service account private key to authenticate clients for GCS API which is bad. It means credentials are deployed within AppEngine and it is really not needed. Second, I am creating signed URL for files inside the GCS. This is also not needed and I could use authenticated URL to expose packages. Last but not least, there is no authentication setup actually. Now that I am thinking of it, I should have put everything in the TL;DR I will create one issue per topic I have to fix and hopefully it will be safer to use I'll try to advance on that this weekend and see if I can deliver a version for next week 😉 |
Haha, this is awesome! So honest! :) I did see your service account requirement, however from what i've read up on GCS in app engine you shouldn't need anything at all (https://cloud.google.com/appengine/docs/standard/go/using-cloud-storage#setting_bucket_and_object_permissions). I think the call to the I guess you could also 'proxy' the packages through the service not requiring anything special from GCS. That might be less than ideal for larger services though I guess. Lastly - sure, the security is a concern! In fact in our use case the server would be for running behind a VPC connected to 'the office' via VPN - nothing exposed over the internet. Cool in any case! Good luck with the project, it's certainly a great case to use go ;) |
I made good progress with fixing the mentionned issue. Well, I went a bit extreme and I took the opportunity to review everything that I did. My work is on branch You can find there the Work is still under progress as I have two problems:
|
@kubaraczkowski the issues I had has been solved with PR #6. Please reopen this issue if you have still trouble deploying, |
Hi,
First of all - great thanks! This is a great little project, exactly what I was looking for - go for quick startup, simple code, GCS backend. Super!
I am trying to setup the gopypi server on App Engine, just as you've suggested. However, the templates seem to have trouble loading. What I get is
It works just fine locally, of course.
I tried running from
cmd/gcp/main.go
as well as copying themain.go
to the root folder.The
app.yaml
is:Could you perhaps share an example app.yaml?
Any other hints ?
Thanks again!
The text was updated successfully, but these errors were encountered: