search

vfarid / v2ray-worker

Total solution for V2Ray over Cloudflare's worker
GNU General Public License v3.0
880 stars 1.21k forks source link

Fix Vless Refused & Update Interface using AdminLTE #14

Closed ozipoetra closed 7 months ago

ozipoetra commented 8 months ago
vfarid commented 7 months ago

Hey, I reverted all your changes bc you did some unaccaptable changes in the code and i didnt realise at first:

  1. You put your avator in the code!!
  2. You added AdminLTE but didnt fix the view so there was many graphical issue
  3. You did critical issue on authentication code so the panel wass accessible without the password.

For the furure, if you wanted to send a pull request, please note above mentioned items and send seperate pull request for each individual change.

Thanks.

ozipoetra commented 7 months ago

Hello, it's okay for that. I will try to answer about that

  1. Sorry for not change the logo before made a pr.
  2. Yeah i know this, it's still not in good state but at least it's better on mobile view. I know maybe it's not reasonable, sorry for this also.
  3. For this you might doesn't set secret KV_NAME and fill like what i write in readme. For reference you can take a look my deployed worker with that commit https://sub.ozi.workers.dev/ also maybe this to make sure https://github.com/ozipoetra/v2ray-worker/actions/runs/6847282666/job/18615273834
vfarid commented 7 months ago

Done. You may make changes for item 3 if needed. i figured out that it has some error in workflow. Thanks