Bug 5887 - JHB unresponsive after enabling password protection in SC7
: JHB unresponsive after enabling password protection in SC7
Status: CLOSED FIXED
Product: SB Controller
Classification: Unclassified
Component: UI
: unspecified
: PC Other
: P1 normal with 1 vote (vote)
: 7.0.1
Assigned To: Richard Titmuss
:
Depends on:
Blocks: 10002
  Show dependency treegraph
 
Reported: 2007-10-22 20:05 UTC by Peter Watkins
Modified: 2008-11-17 14:22 UTC (History)
9 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Watkins 2007-10-22 20:05:16 UTC
r727 cannot cope with SC7 enforcing username/password authentication. Network debugging messages show SC7 replying to Jive HTTP requests with status 401 --which is good and proper-- but there appears to be no way to configure the username & password for the JHB device, even after a factory reset.
Comment 1 Chris Owens 2007-10-30 10:59:08 UTC
Best bug candidate.  Richard asks for Dean's comments on how this should be handled.
Comment 2 Richard Titmuss 2007-11-09 03:08:29 UTC
Dean, please comment on the UI needed.
Comment 3 Blackketter Dean 2007-11-09 15:25:19 UTC
To fix this we'll need to add a sequence of name and password entry screens on Jive when the HTTP request to SC is authenticated.
Comment 4 Peter Watkins 2007-11-12 19:29:57 UTC
A quick glance at SlimDiscoveryApplet, Comet, HttpPool, and RequestHttp suggests that all those classes may need modification to support utilizing HTTP Basic authentication for requests to SC7. 

It looks like most APIs want multiple args to describe a server, e.g.

jive.net.SocketHttp(jnt, "192.168.1.1", 9000, "slimserver")
RequestHttp(sink, 'GET', 'http://192.168.1.1:9000/xml/status.xml')

I think it might be cleaner to refactor those so that instead of host/IP and port, you'd pass objects representing known web servers. Setup would create a web server object in the jnt tables that represents the current SqueezeCenter host, so those method calls would look like

-- nil means use the default web server (SC7 host)
jive.net.SocketHttp(jnt, nil, "slimserver")
-- added an arg. Again, nil means default SC7 host
RequestHttp(sink, 'GET', nil, '/xml/status.xml')

Using references to a new web server class would mean that, going forward, Jive wouldn't have to keep adding new args to HTTP-related methods in order to support additional options (https, timeouts, whatever else might be desired).
Comment 5 Richard Titmuss 2007-12-04 02:14:45 UTC
Punting on this to 7.0
Comment 6 Peter Watkins 2007-12-08 20:24:44 UTC
FYI, the behavior has changed a bit. I had JHB/r1096 working fine with an SC7 install that did not have password auth enabled. I shut that instance down, started SC7 on another host with password auth on, and while the JHB was responsive, it found no players, and I could not use the Music Source settings menu. Just as I was going to file a new bug, I figured it might be related to this one. Disable password auth on SC7 and all's well w/ JHB.
Comment 7 Richard Titmuss 2008-01-11 06:12:31 UTC
It's unlikely this will make the 7.0 release, but we must fix this in 7.0.1.
Comment 8 Richard Titmuss 2008-03-12 04:53:29 UTC
Reset priority
Comment 9 James Richardson 2008-03-13 09:03:08 UTC
*** Bug 7483 has been marked as a duplicate of this bug. ***
Comment 10 KDF 2008-03-17 22:52:03 UTC
*** Bug 7541 has been marked as a duplicate of this bug. ***
Comment 11 Noah Swint 2008-03-18 09:49:28 UTC
Here are some logs from the Duet Receiver if you need them.  Shiksa happens to be the slimserver on my network


Mar 18 12:44:57 (none) user.info jive: (Comet.lua:589) - Comet {shiksa}: _handshake error: HTTP/1.1 401 Unauthorized
Mar 18 12:44:57 (none) user.warn jive: (Comet.lua:890) - Comet {shiksa}: handleAdvice state=CONNECTING
Mar 18 12:44:57 (none) user.info jive: (NetworkThread.lua:246) - NOTIFY cometDisconnected: Comet {shiksa}, 0
Mar 18 12:44:57 (none) user.info jive: (SlimServer.lua:398) - SlimServer {shiksa} disconnected
Mar 18 12:44:57 (none) user.info jive: (NetworkThread.lua:246) - NOTIFY serverDisconnected: SlimServer {shiksa}, 0
Mar 18 12:44:57 (none) user.info jive: (Comet.lua:923) - Comet {shiksa}: advice is retry, connect in 3.626 seconds
Comment 12 Jeff Flowerday 2008-04-03 19:24:31 UTC
What's the status on this one.  I want to enable password protection for remote access, but still need to use my controller.
Comment 13 Richard Titmuss 2008-04-04 02:41:06 UTC
This is on my list of bugs to fix soon.
Comment 14 Richard Titmuss 2008-04-18 12:30:03 UTC
Fixed in jive 7.0.1 r2281.

At the moment this does not check if the username/password have been entered correctly, I have opened bug 7879 to fix this in a future release.
Comment 15 Joe Buccino 2008-04-19 05:06:58 UTC
I'd love to test this but I updated my Controller this morning and got r2269. How can I install r2281?
Comment 16 Joe Buccino 2008-04-20 06:11:12 UTC
r2287 finally showed up and everything seems to be working fine (after struggling for 4 hours with the controller not being able to update).
Comment 17 James Richardson 2008-05-06 11:38:50 UTC
(In reply to comment #16)
> r2287 finally showed up and everything seems to be working fine (after
> struggling for 4 hours with the controller not being able to update).
> 

Marking bug as Closed.
Comment 18 Spies Steven 2008-05-15 08:45:05 UTC
*** Bug 8160 has been marked as a duplicate of this bug. ***