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

Bug: cannot change OPENIM_CHAT_API_PORT #365

Closed
AndrewZuo01 opened this issue Jan 2, 2024 · 8 comments · Fixed by #366
Closed

Bug: cannot change OPENIM_CHAT_API_PORT #365

AndrewZuo01 opened this issue Jan 2, 2024 · 8 comments · Fixed by #366
Labels
bug Categorizes issue or PR as related to a bug. good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.

Comments

@AndrewZuo01
Copy link
Contributor

AndrewZuo01 commented Jan 2, 2024

What happened?

I change OPENIM_CHAT_API_PORT to 40008, then I cannot regsister

What did you expect to happen?

I should been able to register

How can we reproduce it (as minimally and precisely as possible)?

change OPENIM_CHAT_API_PORT to 40008 in .env

Anything else we need to know?

No response

version

```console $ {name} version # paste output here ```

Cloud provider

OS version

```console # On Linux: $ cat /etc/os-release # paste output here $ uname -a # paste output here # On Windows: C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture # paste output here ```

Install tools

@cubxxw
Copy link
Contributor

cubxxw commented Jan 3, 2024

May I kindly inquire where the OPENIM_CHAT_API_PORT variable is modified, and where the modifications are subsequently detected? Additionally, could you provide any error logs for review? Thank you in advance for your assistance.

@AndrewZuo01
Copy link
Contributor Author

image

@kubbot
Copy link
Contributor

kubbot commented Jan 3, 2024

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


image

@cubxxw
Copy link
Contributor

cubxxw commented Jan 3, 2024

@AndrewZuo01 this mysql start error?It seems like you may have posted in the wrong place. Let's double-check and make sure everything is in the right spot. Thank you!

@AndrewZuo01
Copy link
Contributor Author

It's the right logs after changing OPENIM_CHAT_API_PORT

@cubxxw
Copy link
Contributor

cubxxw commented Jan 4, 2024

Due to the fact that the "check_all.sh" script in the chat application has hardcoded the port and name, any adjustments would require a significant amount of effort to modify the code for the chat application. Therefore, I propose that we consider incorporating this adjustment into the future enhancement plan.

@cubxxw cubxxw transferred this issue from openimsdk/openim-docker Jan 4, 2024
@cubxxw cubxxw added good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. bug Categorizes issue or PR as related to a bug. labels Jan 4, 2024
@kubbot
Copy link
Contributor

kubbot commented Jan 4, 2024

This issue is available for anyone to work on. Make sure to reference this issue in your pull request. ✨ Thank you for your contribution! ✨
Join slack 🤖 to connect and communicate with our developers.
If you wish to accept this assignment, please leave a comment in the comments section: /accept.🎯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Categorizes issue or PR as related to a bug. good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants