Skip to content

integrate support for GAE dev#328

Closed
stephenplusplus wants to merge 1 commit intogoogleapis:masterfrom
stephenplusplus:spp--gae
Closed

integrate support for GAE dev#328
stephenplusplus wants to merge 1 commit intogoogleapis:masterfrom
stephenplusplus:spp--gae

Conversation

@stephenplusplus
Copy link
Contributor

Do not merge

RE: #104
Blocked by: #233
Sucessor to: #154

This isn't immediately useful, but is a modernized adaptation of #154.

@stephenplusplus
Copy link
Contributor Author

To do: use the environment's project ID - #365 (comment)

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 2, 2015
@jgeewax
Copy link
Contributor

jgeewax commented Feb 3, 2015

(Added label based on the heading...)

@jgeewax
Copy link
Contributor

jgeewax commented Feb 3, 2015

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?

@ryanseys
Copy link
Contributor

ryanseys commented Feb 3, 2015

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.

@jgeewax
Copy link
Contributor

jgeewax commented Feb 4, 2015

👍 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?

@stephenplusplus stephenplusplus force-pushed the spp--gae branch 2 times, most recently from af4a355 to a11254b Compare February 4, 2015 18:32
@stephenplusplus
Copy link
Contributor Author

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.

@stephenplusplus
Copy link
Contributor Author

#458 will squash this.

@coveralls
Copy link

Coverage Status

Coverage decreased (-39.9%) to 55.204% when pulling 0cf04ec on stephenplusplus:spp--gae into cd9487a on GoogleCloudPlatform:master.

@stephenplusplus
Copy link
Contributor Author

@coveralls you're so weird.

chingor13 pushed a commit that referenced this pull request Aug 22, 2022
For each client method returning a long running operation, a separate method to check its status is added.

Added methods: `checkExportAssetsProgress`, `checkExportIamPolicyAnalysisProgress`.
sofisl pushed a commit that referenced this pull request Sep 16, 2022
sofisl pushed a commit that referenced this pull request Oct 5, 2022
sofisl pushed a commit that referenced this pull request Oct 5, 2022
sofisl pushed a commit that referenced this pull request Nov 18, 2022
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
sofisl pushed a commit that referenced this pull request Nov 30, 2022
…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>
miguelvelezsa pushed a commit that referenced this pull request Jan 28, 2026
sofisl pushed a commit that referenced this pull request Feb 4, 2026
sofisl pushed a commit that referenced this pull request Feb 5, 2026
GautamSharda pushed a commit that referenced this pull request Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. status: blocked Resolving the issue is dependent on other work.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants