Bug 4157 - Server IP address should be displayed along with version information on basic server setup pages
: Server IP address should be displayed along with version information on basic...
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 6.5b3
: Macintosh Other
: P2 normal (vote)
: ---
Assigned To: Dan Sully
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-09-19 18:29 UTC by Blackketter Dean
Modified: 2008-09-15 14:39 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
how about this? (1.58 KB, patch)
2006-09-19 20:19 UTC, KDF
Details | Diff
better version. (1.58 KB, patch)
2006-09-19 20:46 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Blackketter Dean 2006-09-19 18:29:35 UTC
This will help the support guys diagnose network configuration issues.
Comment 1 KDF 2006-09-19 20:19:57 UTC
Created attachment 1559 [details]
how about this?

includes changelog entry
Comment 2 KDF 2006-09-19 20:46:52 UTC
Created attachment 1560 [details]
better version.

this takes into account any binding params given on startup
Comment 3 Blackketter Dean 2006-09-19 21:04:24 UTC
Subject: Re:  Server IP address should be displayed along with version information on basic server setup pages

Thanks for the patch, KDF.

I'm a little confused by the 127.0.0.1 bit.  Under what circumstances  
does that show up?  The main reason here is to help folks diagnose  
network problems, to make sure the player and the server are on the  
same subnet.  Does 127.0.0.1 mean you have no other (external)  
network connection?


Comment 4 KDF 2006-09-19 22:39:53 UTC
actually, i just stole the line from elsewhere: Slim::Utils::Prefs::homeURL.

honestly, in this case, the eventual fallback to 127.0.0.1 would not be useful at all.  if it falls that far, we probably want to see that it's undef.  In my case, hostAddr was already returning 127.0.0.1, which I expect would be the case for most servers.  the --httpaddr command line arg allows binding to one address, which shows up.  I'm not sure how we can get  complete output in the case where the server is currently bound to ALL available addresses.
Comment 5 Dan Sully 2006-09-19 22:46:26 UTC
Added in change 9900

kdf's change to Slim::Buttons::Information was wrong, so the address was showing up as '2'

Since we need to retag 6.5.0, this will be there for that.