Commit 2082849
committed
Remove Context.client_id
The property echoed a non-standard `client_id` key from the request's
`_meta`, which nothing in the SDK or the spec populates, so it was
`None` unless a caller injected the meta key by hand. The name also
collided with the OAuth `client_id` that callers usually mean, and the
line carried a `# pragma: no cover` because it had never been
exercised.
The migration guide covers both replacements: reading the raw
`_meta` key from `ctx.request_context.meta`, and the OAuth client via
`get_access_token().client_id`.1 parent 0cb920f commit 2082849
2 files changed
Lines changed: 21 additions & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
916 | 916 | | |
917 | 917 | | |
918 | 918 | | |
| 919 | + | |
| 920 | + | |
| 921 | + | |
| 922 | + | |
| 923 | + | |
| 924 | + | |
| 925 | + | |
| 926 | + | |
| 927 | + | |
| 928 | + | |
| 929 | + | |
| 930 | + | |
| 931 | + | |
| 932 | + | |
| 933 | + | |
| 934 | + | |
| 935 | + | |
| 936 | + | |
| 937 | + | |
| 938 | + | |
| 939 | + | |
919 | 940 | | |
920 | 941 | | |
921 | 942 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
54 | 54 | | |
55 | 55 | | |
56 | 56 | | |
57 | | - | |
58 | 57 | | |
59 | 58 | | |
60 | 59 | | |
| |||
275 | 274 | | |
276 | 275 | | |
277 | 276 | | |
278 | | - | |
279 | | - | |
280 | | - | |
281 | | - | |
282 | | - | |
283 | | - | |
284 | | - | |
285 | | - | |
286 | | - | |
287 | | - | |
288 | 277 | | |
289 | 278 | | |
290 | 279 | | |
| |||
0 commit comments