dani3l0 / honey

Nice and sweet place for all your self-hosted services.
https://honeyy.vercel.app/
138 stars 3 forks source link

[Suggesting new features] Subnet rules, search engine field, sorting... #4

Open nothing-fr opened 3 months ago

nothing-fr commented 3 months ago

Hello, First of all, this is just a feature suggestion, so please don't take it the wrong way as I really like Honey.

However, do you think it's possible to consider the following feature requests:

It might also be useful to be able to choose which criteria 3rd party verification is based on, for example, I have several sub-domains for my applications and they are considered 3rd party. At present, validation seems to be carried out for applications in a sub-path of the site.

I had to modify the compiled .js to change the behavior so that the verification is based on the domain name.

Before modification :

After modification:

Original code (from compiled js):

function P(s){let e=window.location.protocol=="https:"||window.location.hostname=="localhost",t=window.location.hostname,n=!1,i=!0;if(s.startsWith("https://")?n=!0:["http","https"].includes(s.split("://")[0])||(n=e),i){let o=s.split("://");o.length>1?(o=o[1],o=o.split("/")[0],o=o.split(":")[0],i=!o.includes(t)):i=!1}return{isSecure:n,isThirdParty:i}}

The modified code :

function P(s){let e=window.location.protocol=="https:"||window.location.hostname=="localhost",t = new URL(window.location.href).hostname.split('.').slice(-2).join('.'),n=!1,i=!0;if(s.startsWith("https://")?n=!0:["http","https"].includes(s.split("://")[0])||(n=e),i){let o=s.split("://");o.length>1?(o=o[1],o=o.split("/")[0],o=o.split(":")[0],i=!o.includes(t)):i=!1}return{isSecure:n,isThirdParty:i}}

Thanks for reading, have a nice day!

dani3l0 commented 3 months ago

Hey, thanks for checking out my project :>

All features and improvements sound cool. When it comes to search engines, I'm not a big fan of linking them to our SkyNets, but it always can be made configurable. Oh, and the first feature would require a backend (as for now, honey does not), but I'm thinking to provide one for easier bare-metal deployment (in Go ofc)

I will bring honey back to life later coz I just dropped out of university :~P

Please don't close this issue, those improvements are going to be added sooner or later

nothing-fr commented 3 months ago

Nice :+1: I saw you'd already redone the status in Go too...

I'm using Honey as is for now, no worries, you've got time!

Thanks for taking my suggestions into account.