geekmaster / eyefiserver2

Automatically exported from code.google.com/p/eyefiserver2
0 stars 0 forks source link

WebGUI configuration must be filled with default values #16

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
EyeFi uses the only 59278 port. This parameter should not be configurable and 
must be hardcoded.
There should be notices to mark required and optional parameters.

The main idea is to make configuration idiot proof and simple as possible.

Original issue reported on code.google.com by vilius.s...@gmail.com on 26 Jun 2012 at 6:43

GoogleCodeExporter commented 8 years ago
How about hostname? I do not fully understand why it is needed.
I suggest MAC and upload keys should be prefilled by templates too. i.e. 3 
manufacturer bytes and the rest of zeroes for MAC and 32 zeroes for upload key.

Original comment by duke...@gmail.com on 26 Jun 2012 at 6:49

GoogleCodeExporter commented 8 years ago
I totally agree with you. The whole host section should be removed.

Original comment by vilius.s...@gmail.com on 26 Jun 2012 at 8:41

GoogleCodeExporter commented 8 years ago
In that case, I suppose it is a good idea to clean this values from the config 
and python code aswell.

Original comment by duke...@gmail.com on 27 Jun 2012 at 4:17

GoogleCodeExporter commented 8 years ago
Added default values with the last commit. Should be easier for users now. Host 
name/port removed completely from the Web-GUI, but left untouched in config 
file and python script.

Original comment by duke...@gmail.com on 27 Jun 2012 at 6:41

GoogleCodeExporter commented 8 years ago
BTW, I think MACs are entered in longint representation, not in hex format like 
01:23:45:67:89:ab.
I am not sure why it used at all, but it should be displayed and entered in as 
most MACs - in HEX way.

Original comment by vilius.s...@gmail.com on 27 Jun 2012 at 9:11

GoogleCodeExporter commented 8 years ago
But in Settings.xml '-' is used as a MAC byte seperator, so it is easier to 
copy-paste in that format. Meybe we should leave python script untouched but 
make a processing of useer input in any format and express as most common with 
':'?

Original comment by duke...@gmail.com on 28 Jun 2012 at 4:48

GoogleCodeExporter commented 8 years ago
Fixed MAC-address processing in Web-GUI.

Original comment by duke...@gmail.com on 28 Jun 2012 at 9:41