Hi Will,
Can you try this patch please?

https://github.com/inverse-inc/packetfence/commit/b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff
 
<https://github.com/inverse-inc/packetfence/commit/b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff>

Apply it by downloading it and then using patch, i.e:

# cd /usr/local/pf
# wget 
https://github.com/inverse-inc/packetfence/commit/b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff
 
<https://github.com/inverse-inc/packetfence/commit/b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff>
# patch -p1 < b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff 
<https://github.com/inverse-inc/packetfence/commit/b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff>
# bin/pfcmd service pf restart

Then you can try deleting the source from the GUI and then recreating it again.
If it works we've got ourselves a fix.

Regards,
--
Louis Munro
lmu...@inverse.ca <mailto:lmu...@inverse.ca>  ::  www.inverse.ca 
<http://www.inverse.ca/>
+1.514.447.4918 x125  :: +1 (866) 353-6153 x125
Inverse inc. :: Leaders behind SOGo (www.sogo.nu <http://www.sogo.nu/>) and 
PacketFence (www.packetfence.org <http://www.packetfence.org/>)

> On Aug 14, 2017, at 14:29, Rossing, Will <wross...@css.edu> wrote:
> 
> Hey Louis, Yes I was just going to report that that works, just comma 
> separation, the GUI won't show the list but it still works.    Thanks for 
> your reply!
> 
> On Mon, Aug 14, 2017 at 12:37 PM, Louis Munro <lmu...@inverse.ca 
> <mailto:lmu...@inverse.ca>> wrote:
> Hi Will,
> This looks like a bug from the GUI that saves the list of carriers the wrong 
> way.
> 
> Can you try to change the source to this (manually edit the file):
> 
> 
> [sms]
> description=SMS-based registration
> sms_carriers=100061,100107
> type=SMS
> create_local_account=no
> set_access_level_action=
> local_account_logins=0
> 
> Then run this command:
> 
> # /usr/local/pf/bin/pfcmd configreload hard
> 
> 
> 
> And try again?
> 
> The bug, if that's what it is, is in the code that saves the config.
> So editing the file and reloading it should be a (temporary) workaround.
> 
> Please confirm if this works for you.
> If it does we'll open an issue on GitHub for tracking and issue a maintenance 
> patch.
> 
> 
> Regards,
> --
> Louis Munro
> lmu...@inverse.ca <mailto:lmu...@inverse.ca>  ::  www.inverse.ca 
> <http://www.inverse.ca/>
> +1.514.447.4918 x125 <tel:(514)%20447-4918>  :: +1 (866) 353-6153 x125 
> <tel:(866)%20353-6153>
> Inverse inc. :: Leaders behind SOGo (www.sogo.nu <http://www.sogo.nu/>) and 
> PacketFence (www.packetfence.org <http://www.packetfence.org/>)
> 
>> On Aug 14, 2017, at 12:51, Rossing, Will via PacketFence-users 
>> <packetfence-users@lists.sourceforge.net 
>> <mailto:packetfence-users@lists.sourceforge.net>> wrote:
>> 
>> More info, it works when we only put one SMS provider in the list, if we add 
>> more than one, it gets the exception error.
>> This is how it writes multiple carriers to the config file and seems like it 
>> can't parse it properly or something:
>> 
>> [sms]
>> description=SMS-based registration
>> sms_carriers= <<EOT
>> 100061
>> 100107
>> EOT
>> type=SMS
>> create_local_account=no
>> set_access_level_action=
>> local_account_logins=0
>> 
>> 
>> 
>> One provider works:
>> [sms]
>> description=SMS-based registration
>> sms_carriers=100107
>> type=SMS
>> create_local_account=no
>> set_access_level_action=
>> local_account_logins=0
>> 
>> 
>> On Mon, Aug 14, 2017 at 10:59 AM, Rossing, Will <wross...@css.edu 
>> <mailto:wross...@css.edu>> wrote:
>> Just deploying 7.2 to production and am getting the following when choosing 
>> the sms authentication in the captive portal.
>> 
>> Caught exception in captiveportal::Controller::Root>dynamic_application 
>> "Can't call method "fetchall_arrayref" on an undefined value at 
>> /usr/local/pf/lib/pf/sms_carrier.pm <http://sms_carrier.pm/> line 88."
>> 
>> I swear this worked last week when I put the box in production temporarily.  
>> I've tried removing and adding back in carriers, etc.    Any ideas?   I hate 
>> to have to roll back if I can avoid it!
>> 
>> Thanks
>> 
>> Will
>> 
>> --
>> 
>> 
>> Will Rossing
>> Manager, Network Services  | 218.723.6729 <tel:(218)%20723-6729> | 
>> wross...@css.edu <mailto:wross...@css.edu>
>> 
>> 
>> --
>> 
>> 
>> Will Rossing
>> Manager, Network Services  | 218.723.6729 <tel:(218)%20723-6729> | 
>> wross...@css.edu 
>> <mailto:wross...@css.edu>------------------------------------------------------------------------------
>> Check out the vibrant tech community on one of the world's most
>> engaging tech sites, Slashdot.org <http://slashdot.org/>! 
>> http://sdm.link/slashdot_______________________________________________ 
>> <http://sdm.link/slashdot_______________________________________________>
>> PacketFence-users mailing list
>> PacketFence-users@lists.sourceforge.net 
>> <mailto:PacketFence-users@lists.sourceforge.net>
>> https://lists.sourceforge.net/lists/listinfo/packetfence-users 
>> <https://lists.sourceforge.net/lists/listinfo/packetfence-users>
> 
> 
> 
> 
> --
> 
> 
> Will Rossing
> Manager, Network Services  | 218.723.6729 | wross...@css.edu 
> <mailto:wross...@css.edu>

Attachment: signature.asc
Description: Message signed with OpenPGP

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
PacketFence-users mailing list
PacketFence-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/packetfence-users

Reply via email to