Alignak-monitoring-contrib / alignak-app

Desktop application in system tray for Alignak users.
GNU Affero General Public License v3.0
6 stars 2 forks source link

Remove port in config #311

Closed ddurieux closed 6 years ago

ddurieux commented 6 years ago

In the config interface, if we remove the port, in the settings is keep the : perhaps delete it to have the url without the : at the end ;)

mohierf commented 6 years ago

If port is empty, use 80 for http:// and 443 for https://

algorys commented 6 years ago

@mohierf I think @ddurieux was talking about the case where the backend is reachable with the url (without the port).

So to add the port 80 or 443 would not be really manageable or not wanted.

algorys commented 6 years ago

Done. Reopen if needed :wink:

mohierf commented 6 years ago

If you are using something like http://127.0.0.1 it is the same as http://127.0.0.1:80. So if port is left empty in the input field, consider it is 80 and not an empty value 😉