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.

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

403 on firebase-tictactoe example when trying first move #691

Closed
nczempin opened this issue Jun 6, 2017 · 2 comments
Closed

403 on firebase-tictactoe example when trying first move #691

nczempin opened this issue Jun 6, 2017 · 2 comments
Assignees
Labels
type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@nczempin
Copy link

nczempin commented Jun 6, 2017

In which file did you encounter the issue?

example project at /appengine/firebase-tictactoe

Did you change the file? If so, how?

I changed firebase_config.jspf to the code from the firebase console, as instructed.

Describe the issue

In the example /appengine/firebase-tictactoe, I cannot play a game, because when I click on any of the starting player's empty squares, the request
POST http://localhost:8080/move?gameKey=a3620663-5581-4ec5-8b63-0d955e458e0f
receives a
403 (Forbidden)
result:

send @ jquery.min.js:4
ajax @ jquery.min.js:4
r.(anonymous function) @ jquery.min.js:4
moveInSquare @ main.js:92
dispatch @ jquery.min.js:3
q.handle @ jquery.min.js:3
jquery.min.js:4

I can see some data for players in the firebase console.

My development machine is on Ubuntu-16.04.

I managed to get everything set up and can run "mvn appengine:run". All the tests pass, and the server starts up.

Upon calling localhost:8080, initially I got some errors about missing WEB-INF/view/firebase_config.jspf, which I resolved by manually copying that file to my target/appengine-firebase-tictactoe-1.0-SNAPSHOT/WEB-INF/view folder (from the one in src/). That is probably a separate issue, currently not my concern since I have the workaround.
``java.lang.RuntimeException: java.io.FileNotFoundException: WEB-INF/view/firebase_config.jspf (No such file or directory) at com.example.appengine.firetactoe.FirebaseChannel.<init>(FirebaseChannel.java:98) at com.example.appengine.firetactoe.FirebaseChannel.getInstance(FirebaseChannel.java:72) at com.example.appengine.firetactoe.Game.sendUpdateToUser(Game.java:124) at com.example.appengine.firetactoe.Game.sendUpdateToClients(Game.java:130) at com.example.appengine.firetactoe.TicTacToeServlet.doGet(TicTacToeServlet.java:87) at javax.servlet.http.HttpServlet.service(HttpServlet.java:617)

I also see some warnings about directories:
[INFO] GCLOUD: WARNING: Your working directory, (/home/nczempin/git/java-docs-samples/appengine/firebase-tictactoe) is not equal to your [INFO] GCLOUD: web application root (/home/nczempin/git/java-docs-samples/appengine/firebase-tictactoe/target/appengine-firebase-tictactoe-1.0-SNAPSHOT) [INFO] GCLOUD: You will not be able to access files from your working directory on the production server.
I am not sure if these are relevant to the issue at hand.

@nczempin
Copy link
Author

nczempin commented Jun 7, 2017

I was able to reproduce this on a Windows 10 machine.

When using the GCloud shell there, the had to be moved to a different directory, to WEB-INF/view directly under the one that has the pom.xml (firebase-tictactoe)

@lesv lesv added the type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. label Jun 8, 2017
@kurtisvg kurtisvg self-assigned this Jun 5, 2019
@kurtisvg
Copy link
Contributor

Notice: this issue has been closed because it has been inactive for 732 days. You may reopen this issue if it has been closed in error.

ivanmkc pushed a commit that referenced this issue Nov 3, 2022
…en-plugin to v1.6.9 (#691)

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

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [org.sonatype.plugins:nexus-staging-maven-plugin](http://www.sonatype.com/) ([source](https://github.com/sonatype/nexus-maven-plugins)) | `1.6.8` -> `1.6.9` | [![age](https://badges.renovateapi.com/packages/maven/org.sonatype.plugins:nexus-staging-maven-plugin/1.6.9/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/org.sonatype.plugins:nexus-staging-maven-plugin/1.6.9/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/org.sonatype.plugins:nexus-staging-maven-plugin/1.6.9/compatibility-slim/1.6.8)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/org.sonatype.plugins:nexus-staging-maven-plugin/1.6.9/confidence-slim/1.6.8)](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**: 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 [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-datalabeling).
averikitsch pushed a commit that referenced this issue Nov 9, 2022
…en-plugin to v1.6.9 (#691)

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

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [org.sonatype.plugins:nexus-staging-maven-plugin](http://www.sonatype.com/) ([source](https://github.com/sonatype/nexus-maven-plugins)) | `1.6.8` -> `1.6.9` | [![age](https://badges.renovateapi.com/packages/maven/org.sonatype.plugins:nexus-staging-maven-plugin/1.6.9/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/org.sonatype.plugins:nexus-staging-maven-plugin/1.6.9/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/org.sonatype.plugins:nexus-staging-maven-plugin/1.6.9/compatibility-slim/1.6.8)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/org.sonatype.plugins:nexus-staging-maven-plugin/1.6.9/confidence-slim/1.6.8)](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**: 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 [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-datalabeling).
Shabirmean pushed a commit that referenced this issue Nov 9, 2022
…691)

* chore(deps): update dependency com.google.cloud:libraries-bom to v26

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Shabirmean pushed a commit that referenced this issue Nov 9, 2022
…691)

* chore(deps): update dependency com.google.cloud:libraries-bom to v26

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Shabirmean pushed a commit that referenced this issue Nov 9, 2022
…691)

* chore(deps): update dependency com.google.cloud:libraries-bom to v26

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Shabirmean pushed a commit that referenced this issue Nov 10, 2022
…691)

* chore(deps): update dependency com.google.cloud:libraries-bom to v26

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Shabirmean pushed a commit that referenced this issue Nov 17, 2022
…#691)

[![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://github.com/googleapis/java-core) | `1.94.7` -> `1.94.8` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/compatibility-slim/1.94.7)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/confidence-slim/1.94.7)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

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

### [`v1.94.8`](https://github.com/googleapis/java-core/blob/master/CHANGELOG.md#&#8203;1948-httpswwwgithubcomgoogleapisjava-corecomparev1947v1948-2021-04-23)

[Compare Source](https://github.com/googleapis/java-core/compare/v1.94.7...v1.94.8)

</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.

---

 - [ ] <!-- rebase-check -->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).
Shabirmean pushed a commit that referenced this issue Nov 18, 2022
…#691)

[![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://github.com/googleapis/java-core) | `1.94.7` -> `1.94.8` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/compatibility-slim/1.94.7)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/confidence-slim/1.94.7)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

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

### [`v1.94.8`](https://github.com/googleapis/java-core/blob/master/CHANGELOG.md#&#8203;1948-httpswwwgithubcomgoogleapisjava-corecomparev1947v1948-2021-04-23)

[Compare Source](https://github.com/googleapis/java-core/compare/v1.94.7...v1.94.8)

</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.

---

 - [ ] <!-- rebase-check -->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).
anguillanneuf pushed a commit that referenced this issue Dec 5, 2022
…#691)

[![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://github.com/googleapis/java-core) | `1.94.7` -> `1.94.8` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/compatibility-slim/1.94.7)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/1.94.8/confidence-slim/1.94.7)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

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

### [`v1.94.8`](https://github.com/googleapis/java-core/blob/master/CHANGELOG.md#&#8203;1948-httpswwwgithubcomgoogleapisjava-corecomparev1947v1948-2021-04-23)

[Compare Source](https://github.com/googleapis/java-core/compare/v1.94.7...v1.94.8)

</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.

---

 - [ ] <!-- rebase-check -->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).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

No branches or pull requests

3 participants