integrate support for GAE dev#328
Conversation
cdc3e2a to
79e2dbe
Compare
79e2dbe to
4bb0c71
Compare
|
To do: use the environment's project ID - #365 (comment) |
|
(Added label based on the heading...) |
|
Just out of curiosity -- If I'm in App Engine, and want to talk to another project's datastore, and I do have the keyfile, can I still do that? To put it another way, in App Engine, if I provide nothing to the code and just "hope that it figures it out", I'd want it to write to my App Engine app's datastore. I still want the ability to provide a specific set of credentials and a datastore/dataset ID (right now, this is equivalent to a project ID) and have my code talk outside my current app's dataset. Is that doable? |
|
Just quickly looking at the PR here, it doesn't look like it gives us that option. If it detects that we are on GAE, it will automatically use those credentials to authorize the calls. It should definitely provide the option, i.e. if we provide configuration options, it will use those ALWAYS before defaulting to attempting GAE detection. |
|
👍 to @ryanseys : I think we really need to make sure that code does the same thing across platforms. If I manually configure my connection with a specific key, it should do the same thing on GCE as it does on GAE. It sounds like that isn't the case here. @stephenplusplus : Can you take a look? |
af4a355 to
a11254b
Compare
|
If a user provides configuration details, regardless of environment, that should definitely be honored (good catch!). It was overlooked in my original implementation, but I updated to support custom connection configuration never being precedented by magic. |
a11254b to
2e56a23
Compare
2e56a23 to
0cf04ec
Compare
|
#458 will squash this. |
|
@coveralls you're so weird. |
For each client method returning a long running operation, a separate method to check its status is added. Added methods: `checkExportAssetsProgress`, `checkExportIamPolicyAnalysisProgress`.
Co-authored-by: gcf-merge-on-green[bot] <60162190+gcf-merge-on-green[bot]@users.noreply.github.com> Source-Author: F. Hinkelmann <franziska.hinkelmann@gmail.com> Source-Date: Wed Sep 30 14:13:57 2020 -0400 Source-Repo: googleapis/synthtool Source-Sha: 079dcce498117f9570cebe6e6cff254b38ba3860 Source-Link: googleapis/synthtool@079dcce
…methods and messages related to business context feature docs: Updates copyright message (#328) * feat: Add methods and messages related to starring feature feat: Add methods and messages related to business context feature docs: Updates copyright message PiperOrigin-RevId: 427140868 Source-Link: googleapis/googleapis@0f56b69 Source-Link: googleapis/googleapis-gen@5239850 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNTIzOTg1MGVkOWFiMzYxMjViY2U5MmIzODE1YWVmYjZmZWRlODk2YiJ9 * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Do not merge
RE: #104
Blocked by: #233
Sucessor to: #154
This isn't immediately useful, but is a modernized adaptation of #154.