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

Some fixes and improvements #1997

Merged
merged 9 commits into from
Mar 10, 2024
Merged

Some fixes and improvements #1997

merged 9 commits into from
Mar 10, 2024

Conversation

hamid-gh98
Copy link
Contributor

@hamid-gh98 hamid-gh98 commented Mar 8, 2024

Hello,

  1. Fix access log path (maybe some ppl set other paths for access log)
  2. Fix select options for logs in xray settings by the paths in config.json
  3. refactor api controller
  4. add status code to pureMsgJson to use in future
  5. response with 401 to unauthorize the requests
  6. Fix session timeout by redirecting via axios interceptors

@hamid-gh98 hamid-gh98 changed the title Some fix and improvements Some fixes and improvements Mar 10, 2024
@MHSanaei
Copy link
Owner

MHSanaei commented Mar 10, 2024

Hi Hamid
Thanks for your pr
There are two problems with your changes.
one of them, as you see 
Screenshot 2024-03-10 232019

another one when there is no inbound or iplimit active we get this
WARNING - Access log is set to 'none', check your Xray Configs

@hamid-gh98
Copy link
Contributor Author

Hello, hossein.

thanks for your review.

  1. now we only warning if f2b is installed.
  2. logs paths fixed (the problem was when config log path is set to 'none')

now both is fixed

@MHSanaei MHSanaei merged commit 64a5a9f into MHSanaei:main Mar 10, 2024
Repository owner deleted a comment from FlameFandub Aug 2, 2024
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.

2 participants