Skip to content

Add Logging to README.md. - #1901

Merged
daspecster merged 2 commits into
googleapis:masterfrom
daspecster:add-logging-to-readme
Jun 27, 2016
Merged

Add Logging to README.md.#1901
daspecster merged 2 commits into
googleapis:masterfrom
daspecster:add-logging-to-readme

Conversation

@daspecster

Copy link
Copy Markdown
Contributor

@tseaver LMKWYT and if there's anything I should add.
/cc @dhermes

@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 24, 2016
@daspecsterdaspecster changed the title Add Logging to README.md. closes #1898.Add Logging to README.md.Jun 24, 2016
Comment threadREADME.rst
logger = client.logger('log_name')
logger.log_text("A simple entry") # API call


This comment was marked as spam.

@tseaver

Copy link
Copy Markdown
Contributor

LGTM, even if you decide to ignore my nitpick.

@tseaver

Copy link
Copy Markdown
Contributor

Hmm, maybe flesh the example out to show fetching entries? E.g.:

>>>entries, token=logger.list_entries()
>>>forentryinentries:
... printentry.payloadAsimpleentry

@tseaver

Copy link
Copy Markdown
Contributor

@daspecster reminder to close #1898 after merge (I think having closes #1898 at the end of the first line of the commit message hides it from Github).

@tseaver

Copy link
Copy Markdown
Contributor

LGTM.

@daspecster
daspecster merged commit 5c4db0d into googleapis:masterJun 27, 2016
@daspecster
daspecster deleted the add-logging-to-readme branch June 27, 2016 16:18
@daspecster

Copy link
Copy Markdown
ContributorAuthor

@tseaver looks like the closes #1898 in the commit message worked!

@tseavertseaver mentioned this pull request Jun 29, 2016
parthea added a commit that referenced this pull request Nov 24, 2025
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
parthea pushed a commit that referenced this pull request Mar 2, 2026
Updated `system_tests/system_tests_sync/test_default.py` and `system_tests/system_tests_async/test_default.py` to catch `RefreshError` and check for "invalid_grant" in the error message when running with `authorized_user.json` credentials. This allows the test to pass by confirming the library correctly received the error from the server, acknowledging that the CI credentials are often stale.
This builds on @daniel-sanche' work in [PR #1883](https://togithub.com/googleapis/google-auth-library-python/pull/1883).
Recent changes to `main` appear to have adjusted the response values in these tests and was leading to them failing during CI/CD presubmit tests ([see example below](https://togithub.com/googleapis/google-auth-library-python/pull/1901#issuecomment-3682277336)). This update now prevents those failing tests.
---
*PR created automatically by Jules for task [5370487295365926138](https://jules.google.com/task/5370487295365926138) started by @chalmerlowe*
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@daspecster@tseaver@googlebot