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

fix websocket connection issue in docker makeplane/plane#3195 #3196

Merged

Conversation

abidkhan484
Copy link
Contributor

Since the proxy server is running with Nginx, it is recommended to add the relevant proxy header to use Web Socket. The relevant documentation is shared below.

Reference: Nginx WebSocket Proxying documentation

Copy link
Collaborator

@mguptahub mguptahub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix and revert.

nginx/nginx.conf.template Outdated Show resolved Hide resolved
@abidkhan484 abidkhan484 deleted the fix-websocket-docker-setup branch December 20, 2023 14:21
@abidkhan484 abidkhan484 restored the fix-websocket-docker-setup branch December 20, 2023 14:27
@abidkhan484 abidkhan484 reopened this Dec 20, 2023
Copy link
Collaborator

@mguptahub mguptahub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good to go

@abidkhan484
Copy link
Contributor Author

good to go

Thank you for your thorough review of my pull request. Your insights were incredibly helpful, and I appreciate your time and expertise.

@sriramveeraghanta sriramveeraghanta merged commit f200acc into makeplane:preview Dec 29, 2023
3 checks passed
@abidkhan484 abidkhan484 deleted the fix-websocket-docker-setup branch December 29, 2023 12:05
@sriramveeraghanta sriramveeraghanta added this to the v0.14.2-dev milestone Dec 30, 2023
@abidkhan484 abidkhan484 restored the fix-websocket-docker-setup branch January 1, 2024 10:15
abidkhan484 added a commit to abidkhan484/plane that referenced this pull request Jan 1, 2024
abidkhan484 added a commit to abidkhan484/plane that referenced this pull request Jan 1, 2024
mguptahub added a commit that referenced this pull request Jan 10, 2024
* fix websocket connection issue in docker #3195

* fix websocket connection issue for local env with removing from the prod nginx conf template #319

* fix env substitution issue  of proxy_set_header #3196

* review fixes

---------

Co-authored-by: Manish Gupta <manish@mgupta.me>
sriramveeraghanta pushed a commit that referenced this pull request Jan 22, 2024
* fix websocket connection issue in docker #3195

* fix websocket connection issue for local env with removing from the prod nginx conf template #319
sriramveeraghanta pushed a commit that referenced this pull request Jan 22, 2024
* fix websocket connection issue in docker #3195

* fix websocket connection issue for local env with removing from the prod nginx conf template #319

* fix env substitution issue  of proxy_set_header #3196

* review fixes

---------

Co-authored-by: Manish Gupta <manish@mgupta.me>
sriramveeraghanta pushed a commit that referenced this pull request Jan 22, 2024
* fix websocket connection issue in docker #3195

* fix websocket connection issue for local env with removing from the prod nginx conf template #319

* fix env substitution issue  of proxy_set_header #3196

* review fixes

---------

Co-authored-by: Manish Gupta <manish@mgupta.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug]: Websocket connection to the proxy server failed while set up with docker compose file
3 participants