[Endpoints] Fix local development for Guice sample. by frankyn · Pull Request #909 · GoogleCloudPlatform/java-docs-samples · GitHub

There exists a bug with the DevAppServer and Guice + Basic Scaling that causes the main serving port 8080 to 404 when accessing the sample API endpoint, but works from an instance port when this isn't the correct way to access the API endpoint.

Reported here: b/68948960

Thanks @lesv

🤖 I have created a release *beep* *boop* --- ## [0.124.2](googleapis/java-errorreporting@v0.124.1...v0.124.2) (2022-08-09) ### Dependencies * update dependency com.google.cloud:google-cloud-core to v2.8.3 ([#905](googleapis/java-errorreporting#905)) ([41c601e](googleapis/java-errorreporting@41c601e)) * update dependency com.google.cloud:google-cloud-core to v2.8.4 ([#907](googleapis/java-errorreporting#907)) ([320678e](googleapis/java-errorreporting@320678e)) * update dependency com.google.cloud:google-cloud-core to v2.8.5 ([#908](googleapis/java-errorreporting#908)) ([9a2dbe4](googleapis/java-errorreporting@9a2dbe4)) * update dependency com.google.cloud:google-cloud-core to v2.8.6 ([#911](googleapis/java-errorreporting#911)) ([cb40a3c](googleapis/java-errorreporting@cb40a3c)) * update dependency com.google.cloud:google-cloud-core to v2.8.7 ([#912](googleapis/java-errorreporting#912)) ([014c105](googleapis/java-errorreporting@014c105)) * update dependency com.google.cloud:google-cloud-core to v2.8.8 ([#913](googleapis/java-errorreporting#913)) ([f6fc303](googleapis/java-errorreporting@f6fc303)) * update dependency com.google.cloud:google-cloud-shared-dependencies to v3 ([#909](googleapis/java-errorreporting#909)) ([ea3ac29](googleapis/java-errorreporting@ea3ac29)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

…2.0 (#909) [![WhiteSource 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:libraries-bom](https://togithub.com/GoogleCloudPlatform/cloud-opensource-java) | `24.1.2` -> `24.2.0` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/compatibility-slim/24.1.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/confidence-slim/24.1.2)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-translate).

…909) [![WhiteSource 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://togithub.com/googleapis/java-core) | `2.1.4` -> `2.1.6` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/compatibility-slim/2.1.4)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/confidence-slim/2.1.4)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>googleapis/java-core</summary> ### [`v2.1.6`](https://togithub.com/googleapis/java-core/blob/master/CHANGELOG.md#216-httpswwwgithubcomgoogleapisjava-corecomparev215v216-2021-09-21) [Compare Source](https://togithub.com/googleapis/java-core/compare/v2.1.5...v2.1.6) ### [`v2.1.5`](https://togithub.com/googleapis/java-core/blob/master/CHANGELOG.md#215-httpswwwgithubcomgoogleapisjava-corecomparev214v215-2021-09-21) [Compare Source](https://togithub.com/googleapis/java-core/compare/v2.1.4...v2.1.5) </details> --- ### Configuration 📅 **Schedule**: 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. --- - [ ] If you want to rebase/retry this PR, check this box. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-automl).

…2.0 (#909) [![WhiteSource 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:libraries-bom](https://togithub.com/GoogleCloudPlatform/cloud-opensource-java) | `24.1.2` -> `24.2.0` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/compatibility-slim/24.1.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/confidence-slim/24.1.2)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-translate).

…909) [![WhiteSource 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://togithub.com/googleapis/java-core) | `2.1.4` -> `2.1.6` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/compatibility-slim/2.1.4)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/confidence-slim/2.1.4)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>googleapis/java-core</summary> ### [`v2.1.6`](https://togithub.com/googleapis/java-core/blob/master/CHANGELOG.md#216-httpswwwgithubcomgoogleapisjava-corecomparev215v216-2021-09-21) [Compare Source](https://togithub.com/googleapis/java-core/compare/v2.1.5...v2.1.6) ### [`v2.1.5`](https://togithub.com/googleapis/java-core/blob/master/CHANGELOG.md#215-httpswwwgithubcomgoogleapisjava-corecomparev214v215-2021-09-21) [Compare Source](https://togithub.com/googleapis/java-core/compare/v2.1.4...v2.1.5) </details> --- ### Configuration 📅 **Schedule**: 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. --- - [ ] If you want to rebase/retry this PR, check this box. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-automl).

…2.0 (#909) [![WhiteSource 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:libraries-bom](https://togithub.com/GoogleCloudPlatform/cloud-opensource-java) | `24.1.2` -> `24.2.0` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/compatibility-slim/24.1.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/24.2.0/confidence-slim/24.1.2)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-translate).

…909) [![WhiteSource 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://togithub.com/googleapis/java-core) | `2.1.4` -> `2.1.6` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/compatibility-slim/2.1.4)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.1.6/confidence-slim/2.1.4)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>googleapis/java-core</summary> ### [`v2.1.6`](https://togithub.com/googleapis/java-core/blob/master/CHANGELOG.md#216-httpswwwgithubcomgoogleapisjava-corecomparev215v216-2021-09-21) [Compare Source](https://togithub.com/googleapis/java-core/compare/v2.1.5...v2.1.6) ### [`v2.1.5`](https://togithub.com/googleapis/java-core/blob/master/CHANGELOG.md#215-httpswwwgithubcomgoogleapisjava-corecomparev214v215-2021-09-21) [Compare Source](https://togithub.com/googleapis/java-core/compare/v2.1.4...v2.1.5) </details> --- ### Configuration 📅 **Schedule**: 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. --- - [ ] If you want to rebase/retry this PR, check this box. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-automl).

Frank Natividad added 2 commits November 6, 2017 14:12

Fix bugs in README and remove manual scaling..

472dd7a

Fix java7 version of endpoints frameworks guice sample

f573624

frankyn requested a review from lesv November 6, 2017 22:25

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 6, 2017

Remove welcome file from web.xml

1e8de4c

frankyn removed the request for review from lesv November 6, 2017 23:07

frankyn changed the title [Endpoints] Fix local development for Guice sample. [Endpoints] [WIP] [DO NOT MERGE] Fix local development for Guice sample. Nov 6, 2017

frankyn added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 6, 2017

frankyn changed the title [Endpoints] [WIP] [DO NOT MERGE] Fix local development for Guice sample. [Endpoints] [WIP] Fix local development for Guice sample. Nov 6, 2017

Adding region tags and cleaning up.

b38bfd3

frankyn removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 6, 2017

frankyn changed the title [Endpoints] [WIP] Fix local development for Guice sample. [Endpoints] Fix local development for Guice sample. Nov 6, 2017

frankyn requested a review from lesv November 6, 2017 23:34

lesv approved these changes Nov 6, 2017

View reviewed changes

frankyn merged commit 30b58e8 into master Nov 7, 2017

frankyn deleted the fix-endpoints-guice branch November 7, 2017 00:31

Shabirmean mentioned this pull request Nov 15, 2022

samples: migrate code from googleapis/java-dialogflow #7420

Merged

9 tasks

Shabirmean mentioned this pull request Nov 17, 2022

samples: migrate code from googleapis/java-translate #7443

Closed

9 tasks

Shabirmean mentioned this pull request Nov 17, 2022

samples: migrate code from googleapis/java-automl #7444

Merged

9 tasks

Shabirmean mentioned this pull request Nov 18, 2022

samples: migrate code from googleapis/java-translate #7445

Merged

9 tasks

[Endpoints] Fix local development for Guice sample. by frankyn · Pull Request #909 · GoogleCloudPlatform/java-docs-samples · GitHub

Uh oh!

[Endpoints] Fix local development for Guice sample. - #909

[Endpoints] Fix local development for Guice sample.#909
frankyn merged 4 commits into
masterfrom
fix-endpoints-guice

frankyn commented Nov 6, 2017

Uh oh!

frankyn commented Nov 7, 2017

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

[Endpoints] Fix local development for Guice sample. by frankyn · Pull Request #909 · GoogleCloudPlatform/java-docs-samples · GitHub

Uh oh!

[Endpoints] Fix local development for Guice sample. - #909

[Endpoints] Fix local development for Guice sample.#909frankyn merged 4 commits intomasterGoogleCloudPlatform/java-docs-samples:masterfrom fix-endpoints-guiceGoogleCloudPlatform/java-docs-samples:fix-endpoints-guiceCopy head branch name to clipboard

Conversation

frankyn commented Nov 6, 2017

Uh oh!

frankyn commented Nov 7, 2017

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

[Endpoints] Fix local development for Guice sample.#909
frankyn merged 4 commits into
masterfrom
fix-endpoints-guice