Skip to content
This repository has been archived by the owner on Jan 13, 2020. It is now read-only.

Decrease the load of server by using CDN #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Decrease the load of server by using CDN #120

wants to merge 1 commit into from

Conversation

bclswl0827
Copy link

"jquery.min.js"" in line 45 replaced to "//cdnjs.cloudflare.com/ajax/libs/jquery/3.2.1/jquery.min.js"
"jquery.nanoscroller.min.js" in line 46 replaced to "//cdnjs.cloudflare.com/ajax/libs/jquery.nanoscroller/0.8.7/javascripts/jquery.nanoscroller.min.js"
"nanoscroller.css" in line 47 replaced to "//cdnjs.cloudflare.com/ajax/libs/jquery.nanoscroller/0.8.7/css/nanoscroller.css"
All above is to Decrease the load of server.

"jquery.min.js"" in line 45 replaced to "//cdnjs.cloudflare.com/ajax/libs/jquery/3.2.1/jquery.min.js"
"jquery.nanoscroller.min.js" in line 46 replaced to "//cdnjs.cloudflare.com/ajax/libs/jquery.nanoscroller/0.8.7/javascripts/jquery.nanoscroller.min.js"
"nanoscroller.css" in line 47 replaced to "//cdnjs.cloudflare.com/ajax/libs/jquery.nanoscroller/0.8.7/css/nanoscroller.css"
All above is to Decrease the load of server.
@ha7ilm
Copy link
Owner

ha7ilm commented Jan 9, 2019

Hi,

Thank you for the PR.
Those files are included with OpenWebRX so that installing and using it with the default settings does not reveal the presence of the server to any service provider (including CloudFlare).
Does it worth to cache these anyway? I think what takes a lot of time to load is the photo (that's why most servers load slowly.)

Andras

@bclswl0827
Copy link
Author

It worth... In addition, it's better to let users host their background photos to imgur, host avatar to Gravatar and change the icons to SVG icons ( like Font Awesome ), not the icons in gfx folder, this way, the speed of loading may be faster than before.


In one word, try to make OpenWebRX which is installed in localhost only provide with WebSockets and basical HTML page (index.wrx).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants