θΏ™ζ˜―indexlocζδΎ›ηš„ζœεŠ‘οΌŒδΈθ¦θΎ“ε…₯任何密码
Skip to content

Conversation

@timothycarambat
Copy link
Member

@timothycarambat timothycarambat commented May 17, 2024

Pull Request Type

  • ✨ feat
  • πŸ› fix
  • ♻️ refactor
  • πŸ’„ style
  • πŸ”¨ chore
  • πŸ“ docs

Relevant Issues

resolves #1257

What is in this change?

Patch @agents directive not working when running AnythingLLM Docker in HTTPS mode. The certificate would not apply to the WebSocket connection and therefore upgrading from HTTPS<>WSS would fail as the socket was only WS.

This patch fixes that to where AnythingLLM does not need to be run behind a proxy like Nginx in order to have the full system on SSL on the same cert.

Additional Information

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

@timothycarambat timothycarambat merged commit f140139 into master May 17, 2024
@timothycarambat timothycarambat deleted the 1417-fix-wss-upgrade-ssl branch May 17, 2024 21:03
CrackerCat pushed a commit to CrackerCat/anything-llm that referenced this pull request Jul 31, 2024
* Patch WSS upgrade for manual HTTPS certs

* update comment

* refactor
CrackerCat pushed a commit to CrackerCat/anything-llm that referenced this pull request Aug 1, 2024
* Patch WSS upgrade for manual HTTPS certs

* update comment

* refactor
CrackerCat pushed a commit to CrackerCat/anything-llm that referenced this pull request Aug 2, 2024
* Patch WSS upgrade for manual HTTPS certs

* update comment

* refactor
CrackerCat pushed a commit to CrackerCat/anything-llm that referenced this pull request Aug 3, 2024
* Patch WSS upgrade for manual HTTPS certs

* update comment

* refactor
cabwds pushed a commit to cabwds/anything-llm that referenced this pull request Jul 3, 2025
* Patch WSS upgrade for manual HTTPS certs

* update comment

* refactor
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 not being upgraded with manual HTTP certificate set

2 participants