Skip to content
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.

Sign up for GitHub

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

Jump to bottom

NPE when getting non-existing topic or subscription #987

Closed
mziccard opened this issue May 6, 2016 · 1 comment
Closed

NPE when getting non-existing topic or subscription #987

mziccard opened this issue May 6, 2016 · 1 comment
Assignees
@mziccard
Labels
api: pubsub Issues related to the Pub/Sub API. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@mziccard
Copy link
Contributor

mziccard commented May 6, 2016

DefaultPubSubRpc,get returns a Future such that Future.get() returns null if the resource was not found. In PubSubImpl.

In PubSubImpl transform the result using (Topic/Subscription).fromPbFunction() which do not handle the null case.
@mziccard mziccard added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. api: pubsub Issues related to the Pub/Sub API. labels May 6, 2016
@mziccard mziccard mentioned this issue May 17, 2016
Merged
@mziccard
Copy link
Contributor Author

This was fixed in #1015

github-actions bot pushed a commit that referenced this issue Jul 8, 2022
@gcf-owl-bot
chore: fix minor typos in README and pom templates (#1479) (#987)
cf3d6a1 
Source-Link: googleapis/synthtool@18d4e9b
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:f8374176dc59291f05dd3fec927a9da2cda687a9ef4de32e77f699a2be12ab45

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
github-actions bot pushed a commit that referenced this issue Aug 18, 2022
@renovate-bot
deps: update dependency com.google.cloud:google-cloud-core to v2.8.9 (#…
3b9ae1d 
…987)

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:google-cloud-core](https://github.com/googleapis/java-core) | `2.8.8` -> `2.8.9` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.9/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.9/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.9/compatibility-slim/2.8.8)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.9/confidence-slim/2.8.8)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/java-core</summary>

### [`v2.8.9`](https://github.com/googleapis/java-core/blob/HEAD/CHANGELOG.md#&#8203;289-httpsgithubcomgoogleapisjava-corecomparev288v289-2022-08-16)

[Compare Source](https://github.com/googleapis/java-core/compare/v2.8.8...v2.8.9)

##### Dependencies

-   update dependency com.google.auth:google-auth-library-bom to v1.10.0 ([#&#8203;900](https://github.com/googleapis/java-core/issues/900)) ([c01b47a](https://github.com/googleapis/java-core/commit/c01b47a122a9036f8c594cb583dab3512a257597))
-   update dependency com.google.protobuf:protobuf-bom to v3.21.5 ([#&#8203;901](https://github.com/googleapis/java-core/issues/901)) ([13e2fd6](https://github.com/googleapis/java-core/commit/13e2fd6fbae1d34385ec500aaf4a2b38e19dd6de))
-   update dependency org.threeten:threetenbp to v1.6.1 ([#&#8203;902](https://github.com/googleapis/java-core/issues/902)) ([02dac62](https://github.com/googleapis/java-core/commit/02dac62784a6614291836917b022a32745312157))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-dialogflow).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4xNjEuMCIsInVwZGF0ZWRJblZlciI6IjMyLjE2MS4wIn0=-->
@release-please release-please bot mentioned this issue Aug 24, 2022
Closed
github-actions bot pushed a commit that referenced this issue Sep 19, 2022
@release-please
chore(main): release 3.1.1 (#987)
a0ca1ea 
🤖 I have created a release *beep* *boop*
---


## [3.1.1](googleapis/java-vision@v3.1.0...v3.1.1) (2022-09-16)


### Bug Fixes

* Update IT Test Case Results ([#976](googleapis/java-vision#976)) ([eb7372d](googleapis/java-vision@eb7372d))


### Dependencies

* Update dependency com.google.cloud:google-cloud-core to v2.8.10 ([#977](googleapis/java-vision#977)) ([fe3cb82](googleapis/java-vision@fe3cb82))
* Update dependency com.google.cloud:google-cloud-core to v2.8.11 ([#980](googleapis/java-vision#980)) ([915af18](googleapis/java-vision@915af18))
* Update dependency com.google.cloud:google-cloud-core to v2.8.9 ([#973](googleapis/java-vision#973)) ([9b857a6](googleapis/java-vision@9b857a6))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.2 ([#981](googleapis/java-vision#981)) ([8dd78c8](googleapis/java-vision@8dd78c8))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.3 ([#985](googleapis/java-vision#985)) ([7873226](googleapis/java-vision@7873226))
* Update dependency com.google.cloud:google-cloud-storage to v2.11.3 ([#964](googleapis/java-vision#964)) ([726c31e](googleapis/java-vision@726c31e))
* Update spring.version to v2.7.3 ([#974](googleapis/java-vision#974)) ([d3aac93](googleapis/java-vision@d3aac93))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
suztomo pushed a commit that referenced this issue Feb 1, 2023
@renovate-bot
deps: update dependency com.google.api:gax-bom to v2.19.4 (#987)
d78d04a 
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.api:gax-bom](https://github.com/googleapis/gax-java) | `2.19.2` -> `2.19.4` | [![age](https://badges.renovateapi.com/packages/maven/com.google.api:gax-bom/2.19.4/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.api:gax-bom/2.19.4/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.api:gax-bom/2.19.4/compatibility-slim/2.19.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.api:gax-bom/2.19.4/confidence-slim/2.19.2)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/gax-java</summary>

### [`v2.19.4`](https://github.com/googleapis/gax-java/blob/HEAD/CHANGELOG.md#&#8203;2194-httpsgithubcomgoogleapisgax-javacomparev2193v2194-2022-10-19)

[Compare Source](https://github.com/googleapis/gax-java/compare/v2.19.2...v2.19.4)

##### Bug Fixes

-   Scope the throttling metric to exclude element size calculation ([#&#8203;1835](https://github.com/googleapis/gax-java/issues/1835)) ([0287f83](https://github.com/googleapis/gax-java/commit/0287f83f9cabbb9dff6cb7e8ea3068ccc1ab12d9))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-core).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4yNDAuNSIsInVwZGF0ZWRJblZlciI6IjMyLjI0MC41In0=-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mziccard mziccard

Labels
api: pubsub Issues related to the Pub/Sub API. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mziccard