-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Merge samples for Flex and Std #590
Changes from 3 commits
e42d222
63f087d
ebccfbb
905d597
79fca64
c0de187
787fd0b
63377bc
2fd9d85
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,6 +15,8 @@ | |
runtime: nodejs8 | ||
|
||
# [START env] | ||
# The following env variables may contain sensitive information that grants | ||
# anyone access to your database. Do not add this file to your source control. | ||
env_variables: | ||
SQL_USER: YOUR_SQL_USER | ||
SQL_PASSWORD: YOUR_SQL_PASSWORD | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nit: add a comment here telling users not to add this file to source control (if it has any sensitive passwords in it). P.S: @michaelawyu please apply this change to any other relevant There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not without adding a ton of additional complexity to the sample.
…On Fri, Mar 30, 2018 at 12:42 PM Ace Nassri ***@***.***> wrote:
In appengine/cloudsql/app.standard.yaml
<#590 (comment)>
:
> +#
> +# http://www.apache.org/licenses/LICENSE-2.0
> +#
> +# Unless required by applicable law or agreed to in writing, software
> +# distributed under the License is distributed on an "AS IS" BASIS,
> +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> +# See the License for the specific language governing permissions and
> +# limitations under the License.
> +
> +# [START app_yaml]
> +runtime: nodejs8
> +
> +# [START env]
> +env_variables:
> + SQL_USER: YOUR_SQL_USER
> + SQL_PASSWORD: YOUR_SQL_PASSWORD
Nit: is there a better way to do this so that people don't check passwords
into their codebase if they check in app.yaml? (e.g. Cloud KMS, or
Runtime Configurator API?)
(It's very possible the answer is "No." - in which case we should put a
comment here telling users not to add this file to source control if it has
any sensitive passwords in it.)
/cc @jonparrott <https://github.com/jonparrott>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/GoogleCloudPlatform/nodejs-docs-samples/pull/590/files/ebccfbb9f19ff20e674647177fdb6afc0a010d97#r178361551>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAPUc_3D7FVomP3PSM1MsaRHZHJVtgftks5tjoq7gaJpZM4TBc_9>
.
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,15 +1,17 @@ | ||
# Cloud SQL for MySQL Node.js sample on App Engine flexible environment | ||
# Cloud SQL for PostgresSQL Node.js sample on App Engine flexible environment | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nit: IIRC, it's There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Me and my fat fingers :( It's been fixed |
||
|
||
This sample application shows how to use [Google Cloud SQL[[sql] for [PostgreSQL][postgres] | ||
on Google App Engine. | ||
|
||
App Engine Standard users: See tutorial [ | ||
[App Engine Standard][appengine-std] users: See tutorial [ | ||
Using Cloud SQL for PostgreSQL (App Engine Standard Environment)][std-tutorial] for more information on running and deploying this app. | ||
|
||
App Engine Flexible users: See tutorial [ | ||
[App Engine Flexible][appengine-flex] users: See tutorial [ | ||
Using Cloud SQL for PostgreSQL (App Engine Flexible Environment)][flex-tutorial] for more information on running and deploying this app. | ||
|
||
[sql]: https://cloud.google.com/sql/ | ||
[postgres]: https://www.postgresql.org/download/ | ||
[appengine-flex]: https://cloud.google.com/appengine/docs/flexible/nodejs | ||
[appengine-std]: https://cloud.google.com/appengine/docs/standard/nodejs | ||
[flex-tutorial]: https://cloud.google.com/appengine/docs/flexible/nodejs/using-cloud-sql-postgres | ||
[std-tutorial]: https://cloud.google.com/appengine/docs/standard/nodejs/using-cloud-sql-postgres |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,12 @@ | ||
# Node.js error reporting sample for Google App Engine | ||
|
||
This sample shows how to use Stackdriver Error Reporting on Google App Engine | ||
Node.js Standard Environment and Flexible Environment. | ||
Node.js [Standard Environment][appengine-std] and [Flexible Environment][appengine-flex]. | ||
|
||
## Running locally | ||
|
||
Refer to the [appengine/README.md](../README.md) file for instructions on | ||
running and deploying. | ||
|
||
[appengine-flex]: https://cloud.google.com/appengine/docs/flexible/nodejs | ||
[appengine-std]: https://cloud.google.com/appengine/docs/standard/nodejs |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,6 +14,9 @@ | |
# [START app_yaml] | ||
runtime: nodejs8 | ||
|
||
# The following env variables may contain sensitive information that grants | ||
# anyone access to your Mailjet account. Do not add this file to your source | ||
# control. | ||
env_variables: | ||
MJ_APIKEY_PUBLIC: <your-mj-apikey-public> | ||
MJ_APIKEY_PRIVATE: <your-mj-apikey-private> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should this not be checked into source control? If so, perhaps we should add a comment telling people not to commit this file. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done :) |
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,13 @@ | ||
# Memcached sample for Google App Engine | ||
|
||
This sample demonstrates accessing Memcached from Node.js on | ||
[Google App Engine](https://cloud.google.com/appengine) Node.js Standard | ||
Environment and Flexible Environment. | ||
[Google App Engine](https://cloud.google.com/appengine) Node.js [Standard | ||
Environment][appengine-std] and [Flexible Environment][appengine-flex]. | ||
|
||
## Running locally | ||
|
||
Refer to the [appengine/README.md](../README.md) file for instructions on | ||
running and deploying. | ||
|
||
[appengine-flex]: https://cloud.google.com/appengine/docs/flexible/nodejs | ||
[appengine-std]: https://cloud.google.com/appengine/docs/standard/nodejs |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,6 +15,9 @@ | |
runtime: nodejs8 | ||
|
||
# [START env_variables] | ||
# The following env variables may contain sensitive information that grants | ||
# anyone access to your Sendgrid account. Do not add this file to your source | ||
# control. | ||
env_variables: | ||
SENDGRID_API_KEY: <your-sendgrid-api-key> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ditto to "do not commit this file" comment There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done :) |
||
SENDGRID_SENDER: <your-sendgrid-sender> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In
README.md
, you specifystart
anddeploy
scripts. Make sure to add those (and their respective commands) here, or remove them from yourREADME.md
.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done :)