1.2.0-rc1 generates warning from Kit from webservice

Bug #594973 reported by Matthew Holder
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Xibo
Fix Released
Low
Alex Harrington

Bug Description

I have set up a server from the biela branch, this installs correctly and enables me to create users, layouts etc.

However, when I try and register a display I get the following error:

<errormsg>Undefined index: HTTPS</errormsg>
<errornum>8</errornum>
<errortype>Notice</errortype>
<scriptname>/var/www/testing/xibo-biela/server/lib/app/kit.class.php</scriptname>
<scriptlinenum>422</scriptlinenum>

I think this may be related to bug #589581.

I have tried using the Python client and .NET 1.0.7.

Related branches

Revision history for this message
Alex Harrington (alexharrington) wrote :

I think it's more likely to be the webservice schema version being incremented.

If you pull the python client from https://code.launchpad.net/~alexharrington/xibo/pyclient-1.1.1a2 and then change line 2257 of XiboClient.py to "2" instead of "1" it should then connect up OK:
http://bazaar.launchpad.net/~alexharrington/xibo/pyclient-1.1.1a2/annotate/head:/client/python/XiboClient.py#L2257

Alex

Changed in xibo:
status: New → Won't Fix
status: Won't Fix → Incomplete
Revision history for this message
Alex Harrington (alexharrington) wrote :

There will be updated Windows and Linux builds for 1.2.0-rc1 on the client side - older clients will not be supported.

Revision history for this message
Alex Harrington (alexharrington) wrote :

Can you please try changing lib/app/kit.class.php line 422 as follows:

        if((isset($_SERVER['HTTPS'])) && ($_SERVER['HTTPS'] != ''))
        {
            $fullUrl .= 's';
        }

I think that should remove the warning message that line is generating.

Revision history for this message
Alex Harrington (alexharrington) wrote :

Can you also try:

        if(isset($_SERVER['HTTPS']))
        {
            $fullUrl .= 's';
        }

Which I think should give identical results.

Revision history for this message
Matthew Holder (matt-mattmole) wrote :

Hi Alex,

Yes, I will do this when I get a minute.

Matt

summary: - Unable to register client on 1.2.0rc1 biela branch
+ 1.2.0-rc1 generates warning from Kit from webservice
Changed in xibo:
status: Incomplete → Confirmed
milestone: none → 1.2.0-rc1
assignee: nobody → Alex Harrington (alexharrington)
importance: Undecided → Low
Revision history for this message
Matthew Holder (matt-mattmole) wrote :

Hi Alex,

Your code snippet does indeed stop the warning from appearing in the log section of the management interface

Matt

Revision history for this message
Alex Harrington (alexharrington) wrote :

Thanks Matt - I've pushed the fix in.

Changed in xibo:
status: Confirmed → Fix Committed
Changed in xibo:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.